Task #10336 (closed)
Opened 7 years ago
Closed 6 years ago
proper hashing scheme for FS in model
| Reported by: | bpindelski | Owned by: | bpindelski |
|---|---|---|---|
| Priority: | major | Milestone: | 5.0.0-beta1 |
| Component: | OmeroFs | Version: | n.a. |
| Keywords: | n.a. | Cc: | |
| Resources: | n.a. | Referenced By: | n.a. |
| References: | n.a. | Remaining Time: | n.a. |
| Sprint: | FS Demo 3 |
Description (last modified by bpindelski)
Uploaded images need to use a lighter checksum scheme than SHA-1. A "stateful" class has to wrap around Checksum and Message Digest and provide a consistent tiny API for calculations.
This also necessitates a model change in omero-model\resources\mappings so that new columns get created/renamed if needed.
- Do we want all OriginalFiles? and Pixels moved to the new scheme? Or just Pixels?
All OriginalFile? instances. The OriginalFile? table will have a checksumType and checksumValue column.
Change History (8)
comment:1 Changed 7 years ago by jmoore
- Sprint set to FS Demo 2
comment:2 Changed 7 years ago by bpindelski
- Description modified (diff)
comment:3 Changed 7 years ago by bpindelski
- Owner set to bpindelski
- Status changed from new to accepted
comment:4 Changed 6 years ago by jmoore
- Sprint FS Demo 2 deleted
comment:5 Changed 6 years ago by jmoore
- Summary changed from Bug: proper hashing scheme for FS in model to proper hashing scheme for FS in model
Removing "bug" again. 1 less bug!
comment:6 Changed 6 years ago by jmoore
- Sprint set to FS Demo 3
comment:7 Changed 6 years ago by bpindelski
Closing, as https://github.com/openmicroscopy/openmicroscopy/pull/905 addresses the major point of having code enabling further extensibility. Checksum type has been left as SHA1, but now the change to different types will be easier. Model changes will be captures in a separate ticket.
comment:8 Changed 6 years ago by bpindelski
- Resolution set to fixed
- Status changed from accepted to closed
Note: See
TracTickets for help on using
tickets.
You may also have a look at Agilo extensions to the ticket.
Not included in fs demo 2 tag. Removing from sprint