Project

General

Profile

Actions

bug #8843

open

Revise term caching

Added by Andreas Kohlbecker about 4 years ago. Updated about 4 years ago.

Status:
New
Priority:
New
Assignee:
Category:
taxeditor
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Severity:
normal
Found in Version:

Description

The actual mechanisms of the term caching in the teaxeditor is unclear and undocumented. Several parallel classes and concepts exist whereas it is not clear if these are complementary or redundant.

The documentation in TaxonomicEditorDevelopersGuide only mentions the term caching in CachingHttpInvokerProxyFactoryBean.handleTermRequest(RemoteInvocation invocation, MethodInvocation originalInvocation) but there is also the TermStore class which is supposed to load important terms on application startup.

Open questions and Problems:

  1. Why does CachingHttpInvokerProxyFactoryBean (formerly CdmServiceRequestExecutor, see #8842) distinguish terms from other cdm entities where CdmTransientEntityCacher is caching term internally anyway in the CdmTransientEntityCacher.permanentCache? Some terms would end up being cached in the permanentCache others in the RemoteInvocationTermCacher.termTypeMap
  2. Is the TermStore the fist mechanism mentioned in TaxonomicEditorDevelopersGuide#Term-Loading, or is this something different?

TODO

  1. improve type level documentation
  2. improve and update the TaxonomicEditorDevelopersGuide

Related issues

Related to EDIT - bug #8842: CdmServiceRequestExecutor is not stateleass and neither thread save nor access synchronizedResolvedAndreas Müller

Actions
Actions #1

Updated by Andreas Kohlbecker about 4 years ago

  • Description updated (diff)
Actions #2

Updated by Andreas Kohlbecker about 4 years ago

  • Related to bug #8842: CdmServiceRequestExecutor is not stateleass and neither thread save nor access synchronized added
Actions

Also available in: Atom PDF