Re: [PR] OAK-10866 - Avoid creating an intermediate String when appending a string value to a JsopBuilder. [jackrabbit-oak]

2024-06-10 Thread via GitHub
nfsantos commented on code in PR #1511: URL: https://github.com/apache/jackrabbit-oak/pull/1511#discussion_r1632858732 ## oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/json/JsopBuilder.java: ## @@ -275,6 +277,34 @@ public static String encode(String s) {

Re: [PR] OAK-10866 - Avoid creating an intermediate String when appending a string value to a JsopBuilder. [jackrabbit-oak]

2024-06-10 Thread via GitHub
reschke commented on code in PR #1511: URL: https://github.com/apache/jackrabbit-oak/pull/1511#discussion_r1632842172 ## oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/json/JsopBuilder.java: ## @@ -197,9 +197,11 @@ public JsopBuilder value(boolean value) { *

Re: [PR] OAK-10866 - Avoid creating an intermediate String when appending a string value to a JsopBuilder. [jackrabbit-oak]

2024-06-10 Thread via GitHub
reschke commented on code in PR #1511: URL: https://github.com/apache/jackrabbit-oak/pull/1511#discussion_r1632840377 ## oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/json/JsopBuilder.java: ## @@ -275,6 +277,34 @@ public static String encode(String s) {

Re: [PR] OAK-10866 - Avoid creating an intermediate String when appending a string value to a JsopBuilder. [jackrabbit-oak]

2024-06-10 Thread via GitHub
reschke commented on code in PR #1511: URL: https://github.com/apache/jackrabbit-oak/pull/1511#discussion_r1632839219 ## oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/json/JsopBuilder.java: ## @@ -275,6 +277,34 @@ public static String encode(String s) {

[PR] OAK-10866 - Avoid creating an intermediate String when appending a string value to a JsopBuilder. [jackrabbit-oak]

2024-06-07 Thread via GitHub
nfsantos opened a new pull request, #1511: URL: https://github.com/apache/jackrabbit-oak/pull/1511 Microbenchmarks: Adapted from JsopStreamTest: ``` public static void main(String... args) { for (int k = 0; k < 5; k++) { String s = "Hello \"World\" Hello

Re: [PR] OAK-9455 - Improve oak-run check to allow fast remote consistency checks [jackrabbit-oak]

2024-06-07 Thread via GitHub
dulceanu merged PR #1508: URL: https://github.com/apache/jackrabbit-oak/pull/1508 -- 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:

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-06-07 Thread via GitHub
ionutzpi commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1630786177 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -38,24 +44,58 @@ import

Re: [PR] OAK-10812: DocumentNodeStore#diffManyChildren(...) may produce incorr… [jackrabbit-oak]

2024-06-06 Thread via GitHub
stefan-egli commented on PR #1465: URL: https://github.com/apache/jackrabbit-oak/pull/1465#issuecomment-2152945186 PS: [commented](https://issues.apache.org/jira/browse/OAK-10812?focusedCommentId=17852849=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-17852849) on

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-06-06 Thread via GitHub
stefan-egli commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1629695860 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -38,24 +44,58 @@ import

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-06-06 Thread via GitHub
ionutzpi commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1629470940 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -160,14 +161,14 @@ String getValue() {

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-06-06 Thread via GitHub
ionutzpi commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1629427035 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -160,14 +161,14 @@ String getValue() {

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-06-06 Thread via GitHub
ionutzpi commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1629381461 ## oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyStateTest.java: ## @@ -33,11 +37,16 @@ import static

Re: [PR] OAK-10837: Add documentation for UT/IT NodeStore fixtures [jackrabbit-oak]

2024-06-06 Thread via GitHub
stefan-egli merged PR #1493: URL: https://github.com/apache/jackrabbit-oak/pull/1493 -- 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:

Re: [PR] Issue/oak 8848 [jackrabbit-oak]

2024-06-06 Thread via GitHub
reschke commented on code in PR #1474: URL: https://github.com/apache/jackrabbit-oak/pull/1474#discussion_r1629067216 ## oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/version/VersionEditor.java: ## @@ -145,27 +146,73 @@ public void propertyChanged(PropertyState

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-06-06 Thread via GitHub
reschke commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1628995615 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -160,14 +161,14 @@ String getValue() {

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-06-06 Thread via GitHub
reschke commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1628992660 ## oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyStateTest.java: ## @@ -33,11 +37,16 @@ import static

Re: [PR] OAK-10837: Add documentation for UT/IT NodeStore fixtures [jackrabbit-oak]

2024-06-06 Thread via GitHub
adelina-marian commented on PR #1493: URL: https://github.com/apache/jackrabbit-oak/pull/1493#issuecomment-2151575017 > It looks good to me, but I'm not sure if under `Getting started` is the best place to add it. > > Maybe we could have a new section for Testing, and eventually add

Re: [PR] OAK-10837: Add documentation for UT/IT NodeStore fixtures [jackrabbit-oak]

2024-06-06 Thread via GitHub
adelina-marian commented on code in PR #1493: URL: https://github.com/apache/jackrabbit-oak/pull/1493#discussion_r1628895275 ## oak-doc/src/site/site.xml: ## @@ -152,10 +152,12 @@ under the License. - + + + + Review

Re: [PR] OAK-10837: Add documentation for UT/IT NodeStore fixtures [jackrabbit-oak]

2024-06-06 Thread via GitHub
adelina-marian commented on code in PR #1493: URL: https://github.com/apache/jackrabbit-oak/pull/1493#discussion_r1628893648 ## oak-doc/src/site/markdown/development/nodestore-fixtures.md: ## @@ -0,0 +1,44 @@ + + +Testing using different NodeStore fixtures +--- +

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-06-05 Thread via GitHub
ionutzpi commented on PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#issuecomment-2149520736 Hi @reschke , @Joscorbe, @stefan-egli could we close this PR? @reschke it need only your approval. Thanks -- This is an automated message from the Apache Git Service. To

Re: [PR] OAK-10861: avoid use of Tika WriteOutHandler convenience method removed in Tika 2.x [jackrabbit-oak]

2024-06-05 Thread via GitHub
reschke merged PR #1510: URL: https://github.com/apache/jackrabbit-oak/pull/1510 -- 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:

[PR] OAK-10861: avoid use of Tika WriteOutHandler convenience method removed in Tika 2.x [jackrabbit-oak]

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

Re: [PR] OAK-10451 UserPrincipalProvider may cause many conflicts when under load [jackrabbit-oak]

2024-06-05 Thread via GitHub
anchela commented on PR #1128: URL: https://github.com/apache/jackrabbit-oak/pull/1128#issuecomment-2149051571 @mreutegg , since you were the author of the initial proposal, I would like to get another review from your side to see if the refactoring I made last week, still meets your

Re: [PR] OAK-10860: oak-lucene: update gson test dependency and stop using deprecated APIs [jackrabbit-oak]

2024-06-04 Thread via GitHub
reschke merged PR #1509: URL: https://github.com/apache/jackrabbit-oak/pull/1509 -- 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:

[PR] OAK-10860: oak-lucene: update gson test dependency and stop using deprecated APIs [jackrabbit-oak]

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

Re: [PR] Issue/oak 8848 [jackrabbit-oak]

2024-06-04 Thread via GitHub
stefan-egli commented on PR #1474: URL: https://github.com/apache/jackrabbit-oak/pull/1474#issuecomment-2147389103 Regarding the refactoring of `propertyChanged` : so far I am not seeing the advantage of the refactoring. That code isn't reused elsewhere, it is merely cutting a larger

Re: [PR] Issue/oak 8848 [jackrabbit-oak]

2024-06-04 Thread via GitHub
stefan-egli commented on code in PR #1474: URL: https://github.com/apache/jackrabbit-oak/pull/1474#discussion_r1625889552 ## oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/version/VersionEditor.java: ## @@ -152,20 +153,37 @@ public void propertyChanged(PropertyState

Re: [PR] OAK-10837: Add documentation for UT/IT NodeStore fixtures [jackrabbit-oak]

2024-06-04 Thread via GitHub
stefan-egli commented on code in PR #1493: URL: https://github.com/apache/jackrabbit-oak/pull/1493#discussion_r1625869390 ## oak-doc/src/site/site.xml: ## @@ -152,10 +152,12 @@ under the License. - + + + + Review Comment:

[PR] OAK-9455 - Improve oak-run check to allow fast remote consistency checks [jackrabbit-oak]

2024-06-04 Thread via GitHub
dulceanu opened a new pull request, #1508: URL: https://github.com/apache/jackrabbit-oak/pull/1508 (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

Re: [PR] OAK-10852 : excluding another flaky test combination [jackrabbit-oak]

2024-06-04 Thread via GitHub
stefan-egli merged PR #1507: URL: https://github.com/apache/jackrabbit-oak/pull/1507 -- 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:

[PR] OAK-10852 : excluding another flaky test combination [jackrabbit-oak]

2024-06-04 Thread via GitHub
stefan-egli opened a new pull request, #1507: URL: https://github.com/apache/jackrabbit-oak/pull/1507 (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

Re: [PR] OAK-10852: disable BranchCommitGCTest unmergedAddsThenMergedAddThenUnmergedRemovesChildren for FullGCMode.ORPHANS_EMPTYPROPS_UNMERGED_BC [jackrabbit-oak]

2024-06-04 Thread via GitHub
reschke merged PR #1506: URL: https://github.com/apache/jackrabbit-oak/pull/1506 -- 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:

[PR] OAK-10852: disable BranchCommitGCTest unmergedAddsThenMergedAddThenUnmergedRemovesChildren for FullGCMode.ORPHANS_EMPTYPROPS_UNMERGED_BC [jackrabbit-oak]

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

Re: [PR] OAK-10856: oak-run: TextPopulatorTest uses transitive dependency com.beust.jcommander.internal.Maps [jackrabbit-oak]

2024-06-04 Thread via GitHub
reschke merged PR #1505: URL: https://github.com/apache/jackrabbit-oak/pull/1505 -- 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:

[PR] OAK-10856: oak-run: TextPopulatorTest uses transitive dependency com.beust.jcommander.internal.Maps [jackrabbit-oak]

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

Re: [PR] OAK-10852 : refinement of 46476bcbfc to only ignore flaky mode [jackrabbit-oak]

2024-06-03 Thread via GitHub
stefan-egli merged PR #1504: URL: https://github.com/apache/jackrabbit-oak/pull/1504 -- 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:

Re: [PR] Issue/oak 8848 [jackrabbit-oak]

2024-06-03 Thread via GitHub
shodaaan commented on code in PR #1474: URL: https://github.com/apache/jackrabbit-oak/pull/1474#discussion_r1624611555 ## oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/version/VersionEditor.java: ## @@ -152,20 +153,37 @@ public void propertyChanged(PropertyState

[PR] OAK-10852 : refinement of 46476bcbfc to only ignore flaky mode [jackrabbit-oak]

2024-06-03 Thread via GitHub
stefan-egli opened a new pull request, #1504: URL: https://github.com/apache/jackrabbit-oak/pull/1504 (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

Re: [PR] OAK-10854: Allow oak.findOne to find long path documents. [jackrabbit-oak]

2024-06-03 Thread via GitHub
Joscorbe merged PR #1503: URL: https://github.com/apache/jackrabbit-oak/pull/1503 -- 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:

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-06-03 Thread via GitHub
Joscorbe commented on PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#issuecomment-2145265372 +1 from me. -- 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.

Re: [PR] OAK-10851 - Log START/END/FAIL messages of indexing job phases inside the same package [jackrabbit-oak]

2024-06-03 Thread via GitHub
nfsantos merged PR #1501: URL: https://github.com/apache/jackrabbit-oak/pull/1501 -- 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:

[PR] OAK-10854: Allow oak.findOne to find long path documents. [jackrabbit-oak]

2024-06-03 Thread via GitHub
Joscorbe opened a new pull request, #1503: URL: https://github.com/apache/jackrabbit-oak/pull/1503 This adds a new optional argument to `oak.findOne` method to oak-mongo.js. This new parameter allows to control if the find should be extended to long paths. If the second parameter is

Re: [PR] Issue/oak 8848 [jackrabbit-oak]

2024-06-03 Thread via GitHub
stefan-egli commented on code in PR #1474: URL: https://github.com/apache/jackrabbit-oak/pull/1474#discussion_r1624467573 ## oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/version/VersionEditor.java: ## @@ -152,20 +153,37 @@ public void propertyChanged(PropertyState

Re: [PR] OAK-10853 : temporarily disable flaky test [jackrabbit-oak]

2024-06-03 Thread via GitHub
stefan-egli merged PR #1502: URL: https://github.com/apache/jackrabbit-oak/pull/1502 -- 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:

[PR] OAK-10853 : temporarily disable flaky test [jackrabbit-oak]

2024-06-03 Thread via GitHub
stefan-egli opened a new pull request, #1502: URL: https://github.com/apache/jackrabbit-oak/pull/1502 (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

Re: [PR] Issue/oak 8848 [jackrabbit-oak]

2024-06-03 Thread via GitHub
stefan-egli commented on PR #1474: URL: https://github.com/apache/jackrabbit-oak/pull/1474#issuecomment-2144993219 @shodaaan `MoveVersionableNodeWithNodeRepositoryTest` is missing the license header -- This is an automated message from the Apache Git Service. To respond to the message,

[PR] OAK-10851 - Log START/END/FAIL messages of indexing job phases inside the same package [jackrabbit-oak]

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

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-06-03 Thread via GitHub
ionutzpi commented on PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#issuecomment-2144652941 Hi @Joscorbe and @reschke could you please check the PR and approve it if is fine? -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] add class FlatFileStoreSplitter [jackrabbit-oak]

2024-06-02 Thread via GitHub
github-actions[bot] commented on PR #567: URL: https://github.com/apache/jackrabbit-oak/pull/567#issuecomment-2144136858 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

Re: [PR] Fix indeterminate map ordering that may causes Sweep2Test to fail [jackrabbit-oak]

2024-06-01 Thread via GitHub
github-actions[bot] closed pull request #555: Fix indeterminate map ordering that may causes Sweep2Test to fail URL: https://github.com/apache/jackrabbit-oak/pull/555 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] OAK-10451 UserPrincipalProvider may cause many conflicts when under load [jackrabbit-oak]

2024-05-31 Thread via GitHub
anchela commented on PR #1128: URL: https://github.com/apache/jackrabbit-oak/pull/1128#issuecomment-2142532334 @nscendoni , i had another look at the patch and tried to address the TODO. lets set aside some time to go over the changes again, review if my updates are reasonable, working and

[PR] OAK-10847: query-spi: bump log level to ERROR for use of deprecated slf4j.event.Level method [jackrabbit-oak]

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

Re: [PR] OAK-10847: query-spi: bump log level to ERROR for use of deprecated slf4j.event.Level method [jackrabbit-oak]

2024-05-31 Thread via GitHub
reschke merged PR #1500: URL: https://github.com/apache/jackrabbit-oak/pull/1500 -- 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:

Re: [PR] OAK-10841: log error cause when ES cannot process a query [jackrabbit-oak]

2024-05-31 Thread via GitHub
fabriziofortino merged PR #1492: URL: https://github.com/apache/jackrabbit-oak/pull/1492 -- 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:

Re: [PR] OAK-10846 : ignoring testPartialMergeRootCleanup as it seems to leak … [jackrabbit-oak]

2024-05-30 Thread via GitHub
stefan-egli merged PR #1499: URL: https://github.com/apache/jackrabbit-oak/pull/1499 -- 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:

[PR] OAK-10846 : ignoring testPartialMergeRootCleanup as it seems to leak … [jackrabbit-oak]

2024-05-30 Thread via GitHub
stefan-egli opened a new pull request, #1499: URL: https://github.com/apache/jackrabbit-oak/pull/1499 …a thread -- 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

Re: [PR] OAK-10844 : marking a slow test as ignored for now [jackrabbit-oak]

2024-05-30 Thread via GitHub
stefan-egli commented on code in PR #1498: URL: https://github.com/apache/jackrabbit-oak/pull/1498#discussion_r1620890209 ## oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/VersionGarbageCollectorIT.java: ## @@ -270,7 +270,7 @@ public void tearDown()

Re: [PR] OAK-10844 : marking a slow test as ignored for now [jackrabbit-oak]

2024-05-30 Thread via GitHub
stefan-egli merged PR #1498: URL: https://github.com/apache/jackrabbit-oak/pull/1498 -- 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:

Re: [PR] OAK-10844 : marking a slow test as ignored for now [jackrabbit-oak]

2024-05-30 Thread via GitHub
reschke commented on code in PR #1498: URL: https://github.com/apache/jackrabbit-oak/pull/1498#discussion_r1620849115 ## oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/VersionGarbageCollectorIT.java: ## @@ -270,7 +270,7 @@ public void tearDown()

[PR] OAK-10844 : marking a slow test as ignored for now [jackrabbit-oak]

2024-05-30 Thread via GitHub
stefan-egli opened a new pull request, #1498: URL: https://github.com/apache/jackrabbit-oak/pull/1498 (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

Re: [PR] OAK-10843 : exclude a flaky test combination [jackrabbit-oak]

2024-05-30 Thread via GitHub
stefan-egli merged PR #1497: URL: https://github.com/apache/jackrabbit-oak/pull/1497 -- 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:

[PR] OAK-10843 : exclude a flaky test combination [jackrabbit-oak]

2024-05-30 Thread via GitHub
stefan-egli opened a new pull request, #1497: URL: https://github.com/apache/jackrabbit-oak/pull/1497 (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

Re: [PR] OAK-10807 - Add oak-run diff to Azure repositories [jackrabbit-oak]

2024-05-30 Thread via GitHub
dulceanu merged PR #1496: URL: https://github.com/apache/jackrabbit-oak/pull/1496 -- 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:

[PR] OAK-10807 - Add oak-run diff to Azure repositories [jackrabbit-oak]

2024-05-30 Thread via GitHub
dulceanu opened a new pull request, #1496: URL: https://github.com/apache/jackrabbit-oak/pull/1496 (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

Re: [PR] OAK-10844 : removing some test combinations and adding logging to hel… [jackrabbit-oak]

2024-05-30 Thread via GitHub
stefan-egli merged PR #1494: URL: https://github.com/apache/jackrabbit-oak/pull/1494 -- 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:

[PR] OAK-10836: Improve message 'Another copy of the index update is alrea… [jackrabbit-oak]

2024-05-30 Thread via GitHub
amit-jain opened a new pull request, #1495: URL: https://github.com/apache/jackrabbit-oak/pull/1495 …dy running' Minor: Changed message -- 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

[PR] OAK-10844 : removing some test combinations and adding logging to hel… [jackrabbit-oak]

2024-05-30 Thread via GitHub
stefan-egli opened a new pull request, #1494: URL: https://github.com/apache/jackrabbit-oak/pull/1494 …p narrow down where time is spent -- 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

Re: [PR] Issue/oak 8848 [jackrabbit-oak]

2024-05-29 Thread via GitHub
shodaaan commented on code in PR #1474: URL: https://github.com/apache/jackrabbit-oak/pull/1474#discussion_r1619342145 ## oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/MoveVersionableNodeWithMemoryRepositoryTest.java: ## @@ -0,0 +1,26 @@ +package

[PR] OAK-10837: Add documentation for UT/IT NodeStore fixtures [jackrabbit-oak]

2024-05-29 Thread via GitHub
adelina-marian opened a new pull request, #1493: URL: https://github.com/apache/jackrabbit-oak/pull/1493 Added a new page in the public documentation about different nsfixtures that are available when running unit and integration tests.

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-29 Thread via GitHub
ionutzpi commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1618747446 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -216,7 +264,7 @@ static PropertyState

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-29 Thread via GitHub
ionutzpi commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1618747446 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -216,7 +264,7 @@ static PropertyState

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-29 Thread via GitHub
ionutzpi commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1618747131 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -227,8 +275,8 @@ PropertyState

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-29 Thread via GitHub
ionutzpi commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1618747446 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -216,7 +264,7 @@ static PropertyState

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-29 Thread via GitHub
ionutzpi commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1618747446 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -216,7 +264,7 @@ static PropertyState

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-29 Thread via GitHub
ionutzpi commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1618747131 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -227,8 +275,8 @@ PropertyState

Re: [PR] OAK-10812: DocumentNodeStore#diffManyChildren(...) may produce incorr… [jackrabbit-oak]

2024-05-29 Thread via GitHub
mbaedke commented on PR #1465: URL: https://github.com/apache/jackrabbit-oak/pull/1465#issuecomment-2137187902 @stefan-egli , see ticket for an explanation. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-29 Thread via GitHub
ionutzpi commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1618651613 ## oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyStateTest.java: ## @@ -75,10 +82,97 @@ public void

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-29 Thread via GitHub
ionutzpi commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1618648423 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -227,8 +275,8 @@ PropertyState

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-29 Thread via GitHub
ionutzpi commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1618648931 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -216,7 +264,7 @@ static PropertyState

[PR] OAK-10451 UserPrincipalProvider may cause many conflicts when under load [jackrabbit-oak]

2024-05-29 Thread via GitHub
nscendoni opened a new pull request, #1128: URL: https://github.com/apache/jackrabbit-oak/pull/1128 (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

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-29 Thread via GitHub
stefan-egli commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1618588187 ## oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyStateTest.java: ## @@ -75,10 +82,97 @@ public void

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-29 Thread via GitHub
stefan-egli commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1618581875 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -216,7 +264,7 @@ static PropertyState

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-29 Thread via GitHub
ionutzpi commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1618377176 ## oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyStateTest.java: ## @@ -81,4 +88,83 @@ public void

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-29 Thread via GitHub
ionutzpi commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1618376854 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -227,8 +275,8 @@ PropertyState

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-29 Thread via GitHub
ionutzpi commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1618376527 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -116,7 +155,20 @@ public int count() {

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-29 Thread via GitHub
ionutzpi commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1618372804 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -38,24 +43,58 @@ import

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-29 Thread via GitHub
ionutzpi commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1618298493 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -160,14 +161,14 @@ String getValue() {

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-28 Thread via GitHub
reschke commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1617352712 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -160,14 +161,14 @@ String getValue() {

Re: [PR] OAK-10739 : Provide support for FullGC in Mongo DocumentStore [jackrabbit-oak]

2024-05-28 Thread via GitHub
Joscorbe merged PR #1454: URL: https://github.com/apache/jackrabbit-oak/pull/1454 -- 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:

[PR] OAK-10841: log error cause when ES cannot process a query [jackrabbit-oak]

2024-05-28 Thread via GitHub
fabriziofortino opened a new pull request, #1492: URL: https://github.com/apache/jackrabbit-oak/pull/1492 When ES cannot process a query, the generic message `all shards failed` is printed in the logs. Adding the error cause in case of `ElasticsearchException`s. -- This is an automated

Re: [PR] Issues/oak 10781 [jackrabbit-oak]

2024-05-28 Thread via GitHub
dulceanu merged PR #1485: URL: https://github.com/apache/jackrabbit-oak/pull/1485 -- 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:

Re: [PR] OAK-10739 : Provide support for FullGC in Mongo DocumentStore [jackrabbit-oak]

2024-05-28 Thread via GitHub
stefan-egli commented on PR #1454: URL: https://github.com/apache/jackrabbit-oak/pull/1454#issuecomment-2135150432 @Joscorbe I see NodeDocumentRevisionCleanerTest.testRecentRevisionsArePreserved failing - would it be much effort to fix that before the merge? -- This is an automated

Re: [PR] OAK-6761: Convert oak-blob-plugins to OSGi R7 annotations [jackrabbit-oak]

2024-05-28 Thread via GitHub
mbaedke merged PR #1415: URL: https://github.com/apache/jackrabbit-oak/pull/1415 -- 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:

Re: [PR] OAK-10838: oak-lucene: maven install fails in oak-lucene due to baseline check [jackrabbit-oak]

2024-05-28 Thread via GitHub
reschke merged PR #1491: URL: https://github.com/apache/jackrabbit-oak/pull/1491 -- 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:

[PR] OAK-10838: oak-lucene: maven install fails in oak-lucene due to baseline check [jackrabbit-oak]

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

Re: [PR] Issue/oak 8848 [jackrabbit-oak]

2024-05-28 Thread via GitHub
reschke commented on code in PR #1474: URL: https://github.com/apache/jackrabbit-oak/pull/1474#discussion_r1613082105 ## oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/MoveVersionableNodeWithMemoryRepositoryTest.java: ## @@ -0,0 +1,26 @@ +package

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-28 Thread via GitHub
reschke commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1616858582 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -38,24 +43,58 @@ import

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-28 Thread via GitHub
stefan-egli commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1616856073 ## oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyStateTest.java: ## @@ -81,4 +88,83 @@ public void

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-28 Thread via GitHub
stefan-egli commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1616853051 ## oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyStateTest.java: ## @@ -75,10 +82,97 @@ public void

Re: [PR] OAK-10803 - Modified DocumentPropertyState.java to compress values ba… [jackrabbit-oak]

2024-05-28 Thread via GitHub
stefan-egli commented on code in PR #1473: URL: https://github.com/apache/jackrabbit-oak/pull/1473#discussion_r1616850420 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentPropertyState.java: ## @@ -116,7 +155,20 @@ public int count() {

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