task #9678
closed
Remove Derived- and FieldUnitFacadeCacheStrategies
Added by Andreas Müller almost 3 years ago.
Updated almost 3 years ago.
Description
Cache strategies for FieldUnits and DerivedUnits are currently using the DerivedUnitFacadeCacheStrategy or DerivedUnitFacadeFieldUnitCacheStrategy.
This is problematic because
- these cache strategies are only available in cdmlib-service and therefore can be loaded only with workaround within the model classes.
- for units that have data that are against the facade contract an exception is thrown when trying to compute the titleCaches
We should implement cache strategies within cdmlib-model doing more or less the same as the facade cachestrategies. At some places the code can be much more simple then in the facade as the facade is intened to also be used for writing data which makes it much more complex.
This ticket is also a prerequisit for implementing the "identityCache" or "shortCache" for SpecimenOrObservationBase.
- Related to feature request #5951: Add collector, field number and collection to specimen search label added
- Related to task #9679: Formatter and update script for SpecimenOrObservationBase short cache added
- Related to bug #8517: SpecimenOrObservation: data from protected titleCache missig in portal pages added
- Tags deleted (
formatting)
- Status changed from New to In Progress
Mostly implemented but there is still a test in vaadin that fails due to a lazy initialization exception when calling specimen.getTitleCache().
- Related to bug #9683: Auto-initialize root bean added
- Status changed from In Progress to Resolved
The failing test in vaadin is related to incorrect autoinitialization (#9683) which is fixed now. So this ticket should be resolved and is up for review.
- Status changed from Resolved to Closed
- % Done changed from 50 to 100
I think we can close this ticket now. Also the failing test in dataportal is fixed now (mediaspecimen are shown there in typification context with short label as the MediaSpecimenDefaultCacheStrategy is used there now, which is correct).
However, we need to adapt all SpecimenOrObservationBase cachestrategies in a way that they have a long version and a short version which is related to the "identityCache" (#5952).
If anybody wants to review this ticket, please feel free to reopen.
- Related to bug #7904: DerivedUnitFacadeFieldUnitCacheStrategy.getCollectorAndFieldNumber() creates temporary Team objects which can not be garbage collected. added
- Related to bug #9978: Change of CacheStrategy from DerivedUnitFacade to DerivedUnitDefaultCacheStrategy results in different caches for MediaSpecimen added
Also available in: Atom
PDF