ionutzpi commented on code in PR #1605:
URL: https://github.com/apache/jackrabbit-oak/pull/1605#discussion_r1725155205
##########
oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/NodeDocument.java:
##########
@@ -610,6 +610,9 @@ int purgeUncommittedRevisions(final int clusterId, final
int batchSize,
if (!Utils.isCommitted(commitValue) && r.getClusterId() ==
clusterId && olderThanLastWrittenRootRevPredicate.test(r)) {
uniqueRevisions.add(r);
removeBranchCommit(op, r);
+ if (op.getId().equals(Utils.getIdFromPath("/"))) {
+ removeCommitRoot(op, r);
+ }
Review Comment:
Done. It is fine for me.
##########
oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/BranchTest.java:
##########
@@ -56,6 +56,12 @@ public class BranchTest {
@Rule
public DocumentMKBuilderProvider builderProvider = new
DocumentMKBuilderProvider();
+ @Before
+ public void before() {
Review Comment:
Done. I put it in after method.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
