You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-issues@jackrabbit.apache.org by "Mete Atamel (JIRA)" <ji...@apache.org> on 2012/12/12 13:55:22 UTC
[jira] [Reopened] (OAK-506) Avoid extra MK.nodeExists() calls when
already cached
[ https://issues.apache.org/jira/browse/OAK-506?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Mete Atamel reopened OAK-506:
-----------------------------
I'm not sure if the fix helped too much because when I run tests, I still see lots of calls to nodeExists with the same path and revision (see below for a small snippet of the same path and revision 216). Maybe we don't cache at the right place in oak-core and that's why we need to keep calling nodeExists for the same path and revision?
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
nodeExists--path:/libs/wcm/core/i18n/fr/Checked Password Field @216
> Avoid extra MK.nodeExists() calls when already cached
> -----------------------------------------------------
>
> Key: OAK-506
> URL: https://issues.apache.org/jira/browse/OAK-506
> Project: Jackrabbit Oak
> Issue Type: Improvement
> Components: core
> Reporter: Jukka Zitting
> Assignee: Jukka Zitting
> Priority: Minor
> Fix For: 0.6
>
>
> The KernelNodeState class uses MK.nodeExists() to check for the presence of child nodes when dealing with nodes with more than 1000 children. Currently that check is used regardless of whether the node is already cached, which could be avoided.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira