Project

General

Profile

Actions
Copy link

feature request #5575

closed

DNA Sample requires CacheStrategy

Added by Andreas Müller almost 8 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Priority14
Assignee:
Andreas Müller
Category:
cdmlib
Target version:
Release 5.23
Start date:
Due date:
% Done:

100%

Estimated time:
Severity:
normal

Description

What exactly do we want to have in a generic DnaSample Cache Strategy? Is it possible to have a common strategy? A pure DNA Sample only derived from field unit may have different requirements than a sample in a full derivation hierarchy.

Discuss with users: #4585

See also #5573 and #5581

Related issues

Related to EDIT - feature request #7612: CacheStrategy for MediaSpecimensDuplicateAndreas Müller

Actions
Related to EDIT - task #9455: Merge DnaSample and DerivedUnitNewAndreas Müller

Actions
Related to EDIT - task #9595: Use DerivedUnit or DnaSample for DnaSampleGeneralDetailElementClosedKatja Luther

Actions
Related to EDIT - task #9678: Remove Derived- and FieldUnitFacadeCacheStrategiesClosedAndreas Müller

Actions
Related to EDIT - feature request #5573: Cache Strategy for Media Specimen requiredClosedAndreas Müller

Actions
Actions
Copy link
 #1

Updated by Andreas Müller almost 3 years ago

Actions
Copy link
 #2

Updated by Andreas Müller almost 3 years ago

  • Description updated (diff)
  • Private changed from Yes to No
Actions
Copy link
 #3

Updated by Andreas Müller almost 3 years ago

  • Status changed from New to In Progress
  • Priority changed from New to Priority14
  • Target version changed from Unassigned CDM tickets to Release 5.19
  • % Done changed from 0 to 30

I added a very first version for DNASample cache strategy to avoid the -titleCache not availble- messages

Actions
Copy link
 #4

Updated by Andreas Müller almost 3 years ago

  • Target version changed from Release 5.19 to Release 5.21

As TissueSample are currently still handled as DnaSample modellwise (to allow adding of sequences directly without DnaSample intermediate) we need to distinguish the correct CacheStrategy depending on the SpecimenOrObservationType. This can only be done after constructing the object as otherwise the type has not been filled by hibernate yet for persistent objects.

Therefore I refactored the initialization of cache strategies in the way that they are only loaded when needed: 493cdee0c5

Actions
Copy link
 #5

Updated by Andreas Müller almost 3 years ago

  • Related to task #9455: Merge DnaSample and DerivedUnit added
Actions
Copy link
 #6

Updated by Andreas Müller almost 3 years ago

  • Status changed from In Progress to Resolved
Actions
Copy link
 #7

Updated by Andreas Müller almost 3 years ago

  • Target version changed from Release 5.21 to Release 5.20
Actions
Copy link
 #8

Updated by Andreas Müller over 2 years ago

  • Related to task #9595: Use DerivedUnit or DnaSample for DnaSampleGeneralDetailElement added
Actions
Copy link
 #9

Updated by Andreas Müller over 2 years ago

  • Target version changed from Release 5.20 to Release 5.25
Actions
Copy link
 #10

Updated by Andreas Müller over 2 years ago

  • Status changed from Resolved to Feedback

The initialization of the cache strategy does not seem to fully work yet. See comment in #9595#note-2

Actions
Copy link
 #11

Updated by Andreas Müller over 2 years ago

  • Target version changed from Release 5.25 to Release 5.23
Actions
Copy link
 #12

Updated by Andreas Müller over 2 years ago

  • % Done changed from 30 to 80
Actions
Copy link
 #13

Updated by Andreas Müller over 2 years ago

  • Related to task #9678: Remove Derived- and FieldUnitFacadeCacheStrategies added
Actions
Copy link
 #14

Updated by Andreas Müller over 2 years ago

  • Status changed from Feedback to Closed
  • % Done changed from 80 to 100

Not sure if this is already fully fixed but it needs to be adapted/tested anyway once we unify DnaSample with DerivedUnit class. Also we have an explicit DerivedUnitDefaultCacheStrategy now (#9678) which may improve the situation with cache strategy loading.

So for now I close this ticket.

Actions
Copy link
 #15

Updated by Andreas Müller over 1 year ago

Actions
Copy link

Also available in: Atom PDF