On 01/02/17 09:16, "Marcel Reutegger" <[email protected]> wrote:
>I think in trunk the code path is also a bit different because of >OAK-4528. It may be possible that the issue still exists in trunk, but >does not call diffManyChildren() anymore. > >What happens when you disable the journal diff mechanism in trunk with >-Doak.disableJournalDiff=true ? Good idea, however that alone doesn't let the test fail yet, as both the local diff cache as well as the node children cache avoid diffManyChildren from being used. But if I use brute force and bypass those two caches explicitly - and at the same time increase the test size by increasing # of nodes - then the test fails on trunk too. So indeed trunk seems to avoid this problem as it doesn't go into diffManyChildren for the cases triggered by the test. Cheers, Stefan
