Re: [PR] OAK-10660 : childOrder overwrite-cleanup variant based on getBranch, … [jackrabbit-oak]

2024-02-27 Thread via GitHub
stefan-egli commented on PR #1326: URL: https://github.com/apache/jackrabbit-oak/pull/1326#issuecomment-1966567441 thx for merging @mbaedke ! -- 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 s

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-02-27 Thread via GitHub
stefan-egli commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1504280658 ## oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/GroovyConsole.groovy: ## @@ -124,6 +124,7 @@ class GroovyConsole { new Checkpoi

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-02-27 Thread via GitHub
stefan-egli commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1504283327 ## oak-run/src/main/java/org/apache/jackrabbit/oak/run/RevisionsCommand.java: ## @@ -82,6 +97,7 @@ public class RevisionsCommand implements Command {

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-02-27 Thread via GitHub
stefan-egli commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1504288687 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/NodeDocumentRevisionCleaner.java: ## @@ -0,0 +1,270 @@ +/* + * Licensed to the

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-02-27 Thread via GitHub
stefan-egli commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1504290603 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/VersionGarbageCollector.java: ## @@ -1053,6 +1053,28 @@ private void collectUn

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-02-27 Thread via GitHub
stefan-egli commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1504293588 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/NodeDocumentRevisionCleaner.java: ## @@ -0,0 +1,270 @@ +/* + * Licensed to the

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-02-27 Thread via GitHub
stefan-egli commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1504304416 ## oak-run/src/main/java/org/apache/jackrabbit/oak/run/RevisionsCommand.java: ## @@ -460,6 +494,97 @@ private void sweep(RevisionsOptions options, Closer clo

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-02-27 Thread via GitHub
stefan-egli commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1504309300 ## oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/NodeDocumentRevisionCleanerTest.java: ## @@ -0,0 +1,549 @@ +package org.apache

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-02-27 Thread via GitHub
stefan-egli commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1504314717 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/NodeDocumentRevisionCleaner.java: ## @@ -0,0 +1,270 @@ +/* + * Licensed to the

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-02-27 Thread via GitHub
stefan-egli commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1504315202 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/NodeDocumentRevisionCleaner.java: ## @@ -0,0 +1,270 @@ +/* + * Licensed to the

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-02-27 Thread via GitHub
stefan-egli commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1504318420 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/NodeDocumentRevisionCleaner.java: ## @@ -0,0 +1,270 @@ +/* + * Licensed to the

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-02-27 Thread via GitHub
stefan-egli commented on PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#issuecomment-1966670655 It's generally speaking a bit difficult to review as the code doesn't compile at this point (therefore eg you can't debug through it, to understand it better). As I understand

Re: [PR] OAK-10672: move internal index version in oak-search [jackrabbit-oak]

2024-02-27 Thread via GitHub
fabriziofortino merged PR #1327: URL: https://github.com/apache/jackrabbit-oak/pull/1327 -- 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: dev-unsubscr...

Re: [PR] OAK-10659 : remove orphaned nodes/documents [jackrabbit-oak]

2024-02-27 Thread via GitHub
rishabhdaim merged PR #1315: URL: https://github.com/apache/jackrabbit-oak/pull/1315 -- 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: dev-unsubscr...@jac

Re: [PR] OAK-10659 : remove orphaned nodes/documents [jackrabbit-oak]

2024-02-27 Thread via GitHub
stefan-egli commented on PR #1315: URL: https://github.com/apache/jackrabbit-oak/pull/1315#issuecomment-1966884398 ah, you right away merged it, ok :) -- 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

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-02-27 Thread via GitHub
Joscorbe commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1504533722 ## oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/GroovyConsole.groovy: ## @@ -124,6 +124,7 @@ class GroovyConsole { new CheckpointC

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-02-27 Thread via GitHub
Joscorbe commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1504557325 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/NodeDocumentRevisionCleaner.java: ## @@ -0,0 +1,270 @@ +/* + * Licensed to the Ap

Re: [PR] OAK-10671- [Indexing Job] Improve Mongo regex query: remove condition on non-indexed _path field to speed-up traversal [jackrabbit-oak]

2024-02-27 Thread via GitHub
fabriziofortino commented on code in PR #1331: URL: https://github.com/apache/jackrabbit-oak/pull/1331#discussion_r150551 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/flatfile/pipelined/PipelinedMongoDownloadTask.java: ## @@ -184,65 +181,

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
reschke commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1505586923 ## oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/ManyChildrenIT.java: ## @@ -82,7 +80,7 @@ public void addRemoveNodes() throws Exception { } @T

Re: [PR] OAK-10671- [Indexing Job] Improve Mongo regex query: remove condition on non-indexed _path field to speed-up traversal [jackrabbit-oak]

2024-02-28 Thread via GitHub
nfsantos commented on code in PR #1331: URL: https://github.com/apache/jackrabbit-oak/pull/1331#discussion_r1505604037 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/flatfile/pipelined/PipelinedMongoDownloadTask.java: ## @@ -184,65 +181,70 @@ s

Re: [PR] OAK-10671- [Indexing Job] Improve Mongo regex query: remove condition on non-indexed _path field to speed-up traversal [jackrabbit-oak]

2024-02-28 Thread via GitHub
nfsantos commented on code in PR #1331: URL: https://github.com/apache/jackrabbit-oak/pull/1331#discussion_r1505604811 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/flatfile/pipelined/PipelinedMongoDownloadTask.java: ## @@ -184,65 +181,70 @@ s

[PR] OAK-10676 : used traversed state to find deleted properties [jackrabbit-oak]

2024-02-28 Thread via GitHub
rishabhdaim opened a new pull request, #1332: URL: https://github.com/apache/jackrabbit-oak/pull/1332 (no comment) -- 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 unsubs

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
Joscorbe commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1505968214 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/Commit.java: ## @@ -354,7 +360,27 @@ private void applyToDocumentStore(RevisionVe

Re: [PR] OAK-10670: add support for service principal in oak-upgrade [jackrabbit-oak]

2024-02-28 Thread via GitHub
smiroslav commented on code in PR #1329: URL: https://github.com/apache/jackrabbit-oak/pull/1329#discussion_r1505962367 ## oak-upgrade/src/test/java/org/apache/jackrabbit/oak/upgrade/cli/container/SegmentAzureServicePrincipalNodeStoreContainer.java: ## @@ -0,0 +1,100 @@ +package

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
stefan-egli commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1505997636 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/Commit.java: ## @@ -354,6 +358,39 @@ private void applyToDocumentStore(Revisio

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
reschke commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1506010020 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/Commit.java: ## @@ -354,6 +358,39 @@ private void applyToDocumentStore(RevisionVec

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
Joscorbe commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1506008713 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/Commit.java: ## @@ -354,6 +358,39 @@ private void applyToDocumentStore(RevisionVe

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
reschke commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1506016539 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentNodeStore.java: ## @@ -869,6 +872,11 @@ public int getMemory() {

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
rishabhdaim commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1506046428 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/Commit.java: ## @@ -354,6 +358,39 @@ private void applyToDocumentStore(Revisio

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
rishabhdaim commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1506050953 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentNodeStore.java: ## @@ -571,6 +571,8 @@ public Long apply(@Nullable Str

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
rishabhdaim commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1506056762 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentNodeStore.java: ## @@ -869,6 +872,11 @@ public int getMemory() {

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
rishabhdaim commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1506057283 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentNodeStoreBuilder.java: ## @@ -127,6 +127,7 @@ public class DocumentNod

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
rishabhdaim commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1506058504 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentNodeStoreService.java: ## @@ -181,6 +181,8 @@ public class DocumentNod

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
mbaedke commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1506059960 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentNodeStore.java: ## @@ -571,6 +571,8 @@ public Long apply(@Nullable String

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
mbaedke commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1506060798 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentNodeStoreBuilder.java: ## @@ -127,6 +127,7 @@ public class DocumentNodeSto

Re: [PR] OAK-10676 : used traversed state to find deleted properties [jackrabbit-oak]

2024-02-28 Thread via GitHub
rishabhdaim merged PR #1332: URL: https://github.com/apache/jackrabbit-oak/pull/1332 -- 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: dev-unsubscr...@jac

Re: [PR] OAK-10676 : used traversed state to find deleted properties [jackrabbit-oak]

2024-02-28 Thread via GitHub
rishabhdaim commented on PR #1332: URL: https://github.com/apache/jackrabbit-oak/pull/1332#issuecomment-1969118364 thanks for review @stefan-egli -- 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

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
stefan-egli commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1506066095 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/Commit.java: ## @@ -354,6 +358,39 @@ private void applyToDocumentStore(Revisio

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
stefan-egli commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1506066452 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentNodeStore.java: ## @@ -869,6 +872,11 @@ public int getMemory() {

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
rishabhdaim commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1506077322 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentNodeStore.java: ## @@ -571,6 +571,8 @@ public Long apply(@Nullable Str

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
stefan-egli commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1506094311 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentNodeStore.java: ## @@ -869,6 +872,11 @@ public int getMemory() {

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
stefan-egli commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1506097290 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/Commit.java: ## @@ -354,6 +358,39 @@ private void applyToDocumentStore(Revisio

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
mbaedke commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1506099419 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentNodeStore.java: ## @@ -571,6 +571,8 @@ public Long apply(@Nullable String

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
reschke commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1506101202 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentNodeStore.java: ## @@ -869,6 +872,11 @@ public int getMemory() {

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
reschke commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1506104757 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentNodeStoreService.java: ## @@ -181,6 +181,8 @@ public class DocumentNodeSto

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
reschke commented on code in PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317#discussion_r1506112813 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/Commit.java: ## @@ -354,6 +358,39 @@ private void applyToDocumentStore(RevisionVec

[PR] OAK-10677: examples: update build-helper-maven-plugin to 3.5.0 [jackrabbit-oak]

2024-02-28 Thread via GitHub
reschke opened a new pull request, #1333: URL: https://github.com/apache/jackrabbit-oak/pull/1333 (no comment) -- 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 unsubscrib

Re: [PR] OAK-10677: examples: update build-helper-maven-plugin to 3.5.0 [jackrabbit-oak]

2024-02-28 Thread via GitHub
reschke merged PR #1333: URL: https://github.com/apache/jackrabbit-oak/pull/1333 -- 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: dev-unsubscr...@jackrab

Re: [PR] OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits [jackrabbit-oak]

2024-02-28 Thread via GitHub
mbaedke merged PR #1317: URL: https://github.com/apache/jackrabbit-oak/pull/1317 -- 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: dev-unsubscr...@jackrab

Re: [PR] OAK-10671- [Indexing Job] Improve Mongo regex query: remove condition on non-indexed _path field to speed-up traversal [jackrabbit-oak]

2024-02-28 Thread via GitHub
steffenvan commented on PR #1331: URL: https://github.com/apache/jackrabbit-oak/pull/1331#issuecomment-1969463801 Don't have anything to add LGTM. -- 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

Re: [PR] OAK-10670: add support for service principal in oak-upgrade [jackrabbit-oak]

2024-02-29 Thread via GitHub
smiroslav commented on PR #1329: URL: https://github.com/apache/jackrabbit-oak/pull/1329#issuecomment-1970690588 @t-rana for new files please add license headers ``` [INFO] BUILD FAILURE [INFO] [

[PR] OAK-10678: update gmavenplus plugin to 3.0.2 [jackrabbit-oak]

2024-02-29 Thread via GitHub
reschke opened a new pull request, #1334: URL: https://github.com/apache/jackrabbit-oak/pull/1334 (no comment) -- 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 unsubscrib

Re: [PR] OAK-10678: update gmavenplus plugin to 3.0.2 [jackrabbit-oak]

2024-02-29 Thread via GitHub
reschke merged PR #1334: URL: https://github.com/apache/jackrabbit-oak/pull/1334 -- 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: dev-unsubscr...@jackrab

Re: [PR] OAK-10671- [Indexing Job] Improve Mongo regex query: remove condition on non-indexed _path field to speed-up traversal [jackrabbit-oak]

2024-02-29 Thread via GitHub
nfsantos merged PR #1331: URL: https://github.com/apache/jackrabbit-oak/pull/1331 -- 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: dev-unsubscr...@jackra

Re: [PR] OAK-10670: add support for service principal in oak-upgrade [jackrabbit-oak]

2024-02-29 Thread via GitHub
t-rana commented on code in PR #1329: URL: https://github.com/apache/jackrabbit-oak/pull/1329#discussion_r1508497917 ## oak-upgrade/src/test/java/org/apache/jackrabbit/oak/upgrade/cli/container/SegmentAzureServicePrincipalNodeStoreContainer.java: ## @@ -0,0 +1,100 @@ +package or

[PR] OAK-10683: Update spring-boot dependency to version 2.7.18 [jackrabbit-oak]

2024-03-01 Thread via GitHub
reschke opened a new pull request, #1335: URL: https://github.com/apache/jackrabbit-oak/pull/1335 (no comment) -- 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 unsubscrib

[PR] Issue/oak 10680 [jackrabbit-oak]

2024-03-01 Thread via GitHub
mbaedke opened a new pull request, #1336: URL: https://github.com/apache/jackrabbit-oak/pull/1336 (no comment) -- 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 unsubscrib

[PR] OAK-10684: update maven-shade-plugin version to 3.5.2 [jackrabbit-oak]

2024-03-01 Thread via GitHub
reschke opened a new pull request, #1337: URL: https://github.com/apache/jackrabbit-oak/pull/1337 (no comment) -- 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 unsubscrib

Re: [PR] OAK-10683: Update spring-boot dependency to version 2.7.18 [jackrabbit-oak]

2024-03-01 Thread via GitHub
reschke merged PR #1335: URL: https://github.com/apache/jackrabbit-oak/pull/1335 -- 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: dev-unsubscr...@jackrab

Re: [PR] OAK-10655 Improve error message in NonDefaultMountWriteReportingObserver [jackrabbit-oak]

2024-03-01 Thread via GitHub
kwin merged PR #1311: URL: https://github.com/apache/jackrabbit-oak/pull/1311 -- 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: dev-unsubscr...@jackrabbit

[PR] OAK-10686: parent: add project.build.outputTimestamp property for Reproducable Builds [jackrabbit-oak]

2024-03-01 Thread via GitHub
reschke opened a new pull request, #1338: URL: https://github.com/apache/jackrabbit-oak/pull/1338 (no comment) -- 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 unsubscrib

Re: [PR] OAK-10680: tests failures with com.arakelian/docker-junit-rule [jackrabbit-oak]

2024-03-01 Thread via GitHub
reschke commented on PR #1336: URL: https://github.com/apache/jackrabbit-oak/pull/1336#issuecomment-1973433681 Oh, wow. I'm somewhat concerned by the amount of changes over on the elastic search side. Either these need to be double-checked by the right people (@fabriziofortino ???),

Re: [PR] OAK-10684: update maven-shade-plugin version to 3.5.2 [jackrabbit-oak]

2024-03-01 Thread via GitHub
reschke merged PR #1337: URL: https://github.com/apache/jackrabbit-oak/pull/1337 -- 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: dev-unsubscr...@jackrab

Re: [PR] OAK-9710: Document addition of feature to support custom TTL for direct download [jackrabbit-oak]

2024-03-01 Thread via GitHub
github-actions[bot] commented on PR #509: URL: https://github.com/apache/jackrabbit-oak/pull/509#issuecomment-1974177235 This PR is stale because it has been open 24 months with no activity. Remove stale label or comment or this will be closed in 30 days. -- This is an automated message f

Re: [PR] fix links to javadoc [jackrabbit-oak]

2024-03-02 Thread via GitHub
github-actions[bot] closed pull request #478: fix links to javadoc URL: https://github.com/apache/jackrabbit-oak/pull/478 -- 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 uns

Re: [PR] OAK-10680: tests failures with com.arakelian/docker-junit-rule [jackrabbit-oak]

2024-03-04 Thread via GitHub
mbaedke commented on PR #1336: URL: https://github.com/apache/jackrabbit-oak/pull/1336#issuecomment-1976026683 @reschke, > we really do not want to introduce shaded Guava into 1.22 Yes, obviously. That one sneaked in with the cherry-pick. I'll remove it. -- This is an automat

Re: [PR] OAK-10686: parent: add project.build.outputTimestamp property for Reproducable Builds [jackrabbit-oak]

2024-03-04 Thread via GitHub
reschke merged PR #1338: URL: https://github.com/apache/jackrabbit-oak/pull/1338 -- 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: dev-unsubscr...@jackrab

[PR] OAK-10687: Restore and cleanup SCM information [jackrabbit-oak]

2024-03-04 Thread via GitHub
reschke opened a new pull request, #1339: URL: https://github.com/apache/jackrabbit-oak/pull/1339 (no comment) -- 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 unsubscrib

Re: [PR] OAK-10670: add support for service principal in oak-upgrade [jackrabbit-oak]

2024-03-04 Thread via GitHub
smiroslav merged PR #1329: URL: https://github.com/apache/jackrabbit-oak/pull/1329 -- 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: dev-unsubscr...@jackr

[PR] OAK-10681 - [indexing job] Support custom filters of paths on Mongo [jackrabbit-oak]

2024-03-04 Thread via GitHub
nfsantos opened a new pull request, #1340: URL: https://github.com/apache/jackrabbit-oak/pull/1340 Add property to specify list of paths to be excluded from Mongo download: `oak.indexer.pipelined.mongoCustomExcludedPaths`. Custom path filtering will only be applied if the index includ

Re: [PR] OAK-10662 improve Reproducible Builds [jackrabbit-oak]

2024-03-04 Thread via GitHub
reschke commented on PR #1318: URL: https://github.com/apache/jackrabbit-oak/pull/1318#issuecomment-1976399507 I *believe* these are created by the ancient Felix SCR plugin (see [OAK-6741](https://issues.apache.org/jira/browse/OAK-6741)) -- This is an automated message from the Apache Git

Re: [PR] OAK-10551: o.a.j.o.jcr.osgi.RepositoryManager declares OSGi propertie… [jackrabbit-oak]

2024-03-04 Thread via GitHub
mbaedke commented on PR #1213: URL: https://github.com/apache/jackrabbit-oak/pull/1213#issuecomment-1976441288 @reschke that is very likely -- 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 spe

Re: [PR] OAK-10551: o.a.j.o.jcr.osgi.RepositoryManager declares OSGi propertie… [jackrabbit-oak]

2024-03-04 Thread via GitHub
reschke commented on PR #1213: URL: https://github.com/apache/jackrabbit-oak/pull/1213#issuecomment-1976495050 I agree these are wrong, but they *are* used as names in the generated OSGi metadata. We need to fix them, butI'm not sure we can simply remove (thus rename) them. -- This is an

Re: [PR] OAK-10687: Restore and cleanup SCM information [jackrabbit-oak]

2024-03-04 Thread via GitHub
reschke merged PR #1339: URL: https://github.com/apache/jackrabbit-oak/pull/1339 -- 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: dev-unsubscr...@jackrab

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-03-04 Thread via GitHub
Joscorbe closed pull request #1191: OAK-10535: Clean up old revisions in a document URL: https://github.com/apache/jackrabbit-oak/pull/1191 -- 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 specif

Re: [PR] OAK-10681 - [indexing job] Support custom filters of paths on Mongo [jackrabbit-oak]

2024-03-04 Thread via GitHub
fabriziofortino commented on code in PR #1340: URL: https://github.com/apache/jackrabbit-oak/pull/1340#discussion_r1511168284 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/flatfile/pipelined/PipelinedMongoDownloadTask.java: ## @@ -126,6 +126,1

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-03-04 Thread via GitHub
Joscorbe commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1511258728 ## oak-run/src/main/java/org/apache/jackrabbit/oak/run/RevisionsCommand.java: ## @@ -82,6 +97,7 @@ public class RevisionsCommand implements Command {

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-03-04 Thread via GitHub
Joscorbe commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1511333001 ## oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/VersionGarbageCollectorIT.java: ## Review Comment: I haven't done any change

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-03-04 Thread via GitHub
Joscorbe commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1511343112 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/NodeDocumentRevisionCleaner.java: ## @@ -0,0 +1,270 @@ +/* + * Licensed to the Ap

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-03-04 Thread via GitHub
Joscorbe commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1511343772 ## oak-run/src/main/java/org/apache/jackrabbit/oak/run/RevisionsCommand.java: ## @@ -460,6 +494,97 @@ private void sweep(RevisionsOptions options, Closer closer

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-03-04 Thread via GitHub
Joscorbe commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1511344989 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/NodeDocumentRevisionCleaner.java: ## @@ -0,0 +1,270 @@ +/* + * Licensed to the Ap

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-03-04 Thread via GitHub
rishabhdaim commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1511357714 ## oak-run/src/main/java/org/apache/jackrabbit/oak/run/RevisionsCommand.java: ## @@ -32,6 +37,7 @@ import java.util.concurrent.TimeUnit; import java.util.c

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-03-04 Thread via GitHub
Joscorbe commented on PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#issuecomment-1976895834 I have squashed the WIP commits, sorry for the forced-PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-03-04 Thread via GitHub
rishabhdaim commented on PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#issuecomment-1976901217 Could you please revert `VersionGarbageCollectorIT` from your PR, it looks like you haven't made any changes in this file. -- This is an automated message from the Apache Git

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-03-04 Thread via GitHub
Joscorbe commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1511389602 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/NodeDocumentRevisionCleaner.java: ## @@ -0,0 +1,270 @@ +/* + * Licensed to the Ap

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-03-04 Thread via GitHub
Joscorbe commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1511417455 ## oak-run/src/main/java/org/apache/jackrabbit/oak/run/RevisionsCommand.java: ## @@ -23,6 +23,11 @@ import java.io.IOException; import java.util.List; import

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-03-05 Thread via GitHub
stefan-egli commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1512625559 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/VersionGarbageCollector.java: ## @@ -1238,12 +1240,11 @@ private void removeUn

Re: [PR] OAK-10680: tests failures with com.arakelian/docker-junit-rule [jackrabbit-oak]

2024-03-05 Thread via GitHub
mbaedke merged PR #1336: URL: https://github.com/apache/jackrabbit-oak/pull/1336 -- 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: dev-unsubscr...@jackrab

[PR] OAK-10691: remove use of Guava Charsets class [jackrabbit-oak]

2024-03-05 Thread via GitHub
reschke opened a new pull request, #1342: URL: https://github.com/apache/jackrabbit-oak/pull/1342 This is a proof-of-concept for the removal of an easy-to-replace use of Guava. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[PR] OAK-10339: Migrate to SLF4J 2.x [jackrabbit-oak]

2024-03-05 Thread via GitHub
mbaedke opened a new pull request, #1343: URL: https://github.com/apache/jackrabbit-oak/pull/1343 Moving to slf4j-2.0.12 and logback-1.5.2. -- 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 spe

Re: [PR] OAK-10551: o.a.j.o.jcr.osgi.RepositoryManager declares OSGi propertie… [jackrabbit-oak]

2024-03-05 Thread via GitHub
mbaedke merged PR #1213: URL: https://github.com/apache/jackrabbit-oak/pull/1213 -- 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: dev-unsubscr...@jackrab

Re: [PR] OAK-6765: Convert oak-jcr to OSGi R7 annotations [jackrabbit-oak]

2024-03-05 Thread via GitHub
mbaedke merged PR #1214: URL: https://github.com/apache/jackrabbit-oak/pull/1214 -- 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: dev-unsubscr...@jackrab

Re: [PR] OAK-10681 - [indexing job] Support custom filters of paths on Mongo [jackrabbit-oak]

2024-03-05 Thread via GitHub
nfsantos merged PR #1340: URL: https://github.com/apache/jackrabbit-oak/pull/1340 -- 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: dev-unsubscr...@jackra

[PR] OAK-10690: Update docs and test config for recommended MongoDB version [jackrabbit-oak]

2024-03-05 Thread via GitHub
reschke opened a new pull request, #1344: URL: https://github.com/apache/jackrabbit-oak/pull/1344 (no comment) -- 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 unsubscrib

[PR] OAK-10674: DocumentStore: verify that we could use Oak's Bloom filter [jackrabbit-oak]

2024-03-05 Thread via GitHub
reschke opened a new pull request, #1345: URL: https://github.com/apache/jackrabbit-oak/pull/1345 (temporary playground for testing) -- 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 c

Re: [PR] OAK-10690: Update docs and test config for recommended MongoDB version [jackrabbit-oak]

2024-03-06 Thread via GitHub
reschke merged PR #1344: URL: https://github.com/apache/jackrabbit-oak/pull/1344 -- 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: dev-unsubscr...@jackrab

Re: [PR] OAK-10690: Update docs and test config for recommended MongoDB version [jackrabbit-oak]

2024-03-06 Thread via GitHub
reschke commented on PR #1344: URL: https://github.com/apache/jackrabbit-oak/pull/1344#issuecomment-1980412843 trunk: [b1b7c5a472|https://github.com/apache/jackrabbit-oak/commit/b1b7c5a472d41116b7d5b85330b0740a44c4e1f7] -- This is an automated message from the Apache Git Service. To respo

[PR] OAK-6769: Convert oak-search-mt to OSGi R7 annotations [jackrabbit-oak]

2024-03-06 Thread via GitHub
mbaedke opened a new pull request, #1346: URL: https://github.com/apache/jackrabbit-oak/pull/1346 Done. -- 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-ma

Re: [PR] OAK-6769: Convert oak-search-mt to OSGi R7 annotations [jackrabbit-oak]

2024-03-06 Thread via GitHub
mbaedke merged PR #1346: URL: https://github.com/apache/jackrabbit-oak/pull/1346 -- 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: dev-unsubscr...@jackrab

Re: [PR] OAK-10535: Clean up old revisions in a document [jackrabbit-oak]

2024-03-06 Thread via GitHub
Joscorbe commented on code in PR #1328: URL: https://github.com/apache/jackrabbit-oak/pull/1328#discussion_r1514423969 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/VersionGarbageCollector.java: ## @@ -1398,6 +1399,24 @@ private void delayOnModif

<    2   3   4   5   6   7   8   9   10   11   >