[ 
https://issues.apache.org/jira/browse/SLING-8481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16861954#comment-16861954
 ] 

Carsten Ziegeler commented on SLING-8481:
-----------------------------------------

I'm not really worried about the size.
Your point about identity is valid though - however, this is cache, therefore a 
feature with this cache information is exactly the same feature as the one 
without the cache information as everything inside the cache can be rebuilt 
using the information from the feature - or in other words if tooling sees a 
feature it will always check whether the cache information is available in the 
feature and use it if available - but it will build the information if it is 
not available.
So we don't have to generate this information up front, it can be generated 
when needed and simply added to the feature.
We could mark extensions which contain cached information in some way, by using 
a prefix in the name or something

A separate file always creates problems as you need to find it and associate it 
with a feature.

> Caching of Feature Model Analyser Metadata
> ------------------------------------------
>
>                 Key: SLING-8481
>                 URL: https://issues.apache.org/jira/browse/SLING-8481
>             Project: Sling
>          Issue Type: Improvement
>          Components: Feature Model
>            Reporter: Dominik Süß
>            Priority: Major
>
> The feature model analyser is currently acting on the information of the 
> feature model files as well as the metadata of artifacts. While feature 
> models have a small footprint the extraction of the metadata from binaries is 
> costly.
> Caching of this metadata should be improved in 2 ways:
> a) metadata should only be extracted once and be cached in process to avoid 
> repetitive reading of binaries to access the same metadta
> b) it should be possible to store a "sidecar" artifact to a feature model to 
> contain the extracted metadata from the binaries it references to eliminate 
> the need for full binaries to be parsed or potentially even to have to be 
> present for the analyser to run.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to