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

Michael Dürig commented on OAK-3568:
------------------------------------

>From looking at the code the following is suspicious:

{code}
            for(Entry<RecordId, RecordId> e: pcm.getValue()){
                try {
                    rua.analyseNode(e.getKey());
                    rua.analyseNode(e.getValue());
                } catch (IllegalStateException ex) {
                    ex.printStackTrace();
                }
            }
{code}

The record ids passed into {{analyseNode}} seem to identify maps, not nodes. So 
I would expect an exception here. I think you should instead be calling 
{{analyseMap}}, which is private ATM. However my current work on OAK-3560 might 
change things with {{RecordUsageAnalyser}} anyway (I probably introduce a more 
general purpose segment parser). 

> Oak Explorer: add persisted compaction map size
> -----------------------------------------------
>
>                 Key: OAK-3568
>                 URL: https://issues.apache.org/jira/browse/OAK-3568
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: run
>            Reporter: Alex Parvulescu
>            Assignee: Alex Parvulescu
>            Priority: Minor
>         Attachments: OAK-3568.patch
>
>
> I'd like to add the persisted compaction map size report



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to