[jira] [Resolved] (OAK-4601) In case of branch commits, nodeCache gets entries for branch revision without "br" prefix

["Vikas Saurabh (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/OAK-4601?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Vikas Saurabh resolved OAK-4601.
--------------------------------
    Resolution: Not A Problem

Further investigation showed that a test like \[0] gets 1 branch with base={{rBase}}, 1commit={rCommit}}. The cache entries in node cache show up for
# {{childX\@brCommit}}
# {{/\@brCommit}}
# {{/\@rBase}}
# {{/\@brBase}}

Technically, #3 and #4 carry the same information from node state pov and could potentially be avoided. But, that's too minor a space waste to worry about.

\[0]:
{code}
@Test
public void nodeCacheForBranchCommit() throws Exception {
    DocumentNodeStore ns = builderProvider.newBuilder().getNodeStore();
    NodeBuilder b1 = ns.getRoot().builder();
    final int NUM_CHILDREN = 3*DocumentRootBuilder.UPDATE_LIMIT + 1;
    //this would push cache entries -> #nodes=3*update_limit
    for (int i = 0; i < NUM_CHILDREN; i++) {
        b1.child("child" + i);
    }
    //this would pull in cache entries
    for (int i = 0; i < NUM_CHILDREN; i++) {
        b1.getChildNode("child" + i);
    }
}
{code}

> In case of branch commits, nodeCache gets entries for branch revision without "br" prefix
> -----------------------------------------------------------------------------------------
>
>                 Key: OAK-4601
>                 URL: https://issues.apache.org/jira/browse/OAK-4601
>             Project: Jackrabbit Oak
>          Issue Type: Bug
>            Reporter: Vikas Saurabh
>            Assignee: Vikas Saurabh
>            Priority: Minor
>
> Similar to OAK-4600, nodeCache also gets cached entries without {{br}} prefix. Still investigating when/where are those entries created.
> Btw, unlike, OAK-4600, this affects trunk too.



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