Task #10336 (closed)
Opened 11 years ago
Closed 11 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 11 years ago by jmoore
- Sprint set to FS Demo 2
comment:2 Changed 11 years ago by bpindelski
- Description modified (diff)
comment:3 Changed 11 years ago by bpindelski
- Owner set to bpindelski
- Status changed from new to accepted
comment:4 Changed 11 years ago by jmoore
- Sprint FS Demo 2 deleted
comment:5 Changed 11 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 11 years ago by jmoore
- Sprint set to FS Demo 3
comment:7 Changed 11 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 11 years ago by bpindelski
- Resolution set to fixed
- Status changed from accepted to closed
Not included in fs demo 2 tag. Removing from sprint