[GitHub] [jackrabbit-oak] nfsantos commented on a diff in pull request #1094: OAK-10424 add support for INSECURE RESULT SIZE and INSECURE FACETS query options and rep:insecureQueryOptions privilege

2023-09-18 Thread via GitHub
nfsantos commented on code in PR #1094: URL: https://github.com/apache/jackrabbit-oak/pull/1094#discussion_r1329183290 ## oak-lucene/src/test/java/org/apache/jackrabbit/oak/jcr/query/ResultSizeTest.java: ## @@ -126,5 +151,90 @@ private void doTestResultSize(boolean union, int

[GitHub] [jackrabbit-oak] adamcin commented on pull request #1094: OAK-10424 add support for INSECURE RESULT SIZE and INSECURE FACETS query options and rep:insecureQueryOptions privilege

2023-09-18 Thread via GitHub
adamcin commented on PR #1094: URL: https://github.com/apache/jackrabbit-oak/pull/1094#issuecomment-1724298220 @thomasmueller I don't have commit rights, so if you are willing to merge it, please do. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [jackrabbit-oak] tihom88 commented on a diff in pull request #1123: OAK-10429: Implementation for IncrementalFFS

2023-09-21 Thread via GitHub
tihom88 commented on code in PR #1123: URL: https://github.com/apache/jackrabbit-oak/pull/1123#discussion_r1333888606 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/incrementalstore/IncrementalFlatFileStoreStrategy.java: ## @@ -0,0 +1,181 @@

[GitHub] [jackrabbit-oak] amit-jain commented on a diff in pull request #1123: OAK-10429: Implementation for IncrementalFFS

2023-09-20 Thread via GitHub
amit-jain commented on code in PR #1123: URL: https://github.com/apache/jackrabbit-oak/pull/1123#discussion_r1331505640 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/DocumentStoreIndexerBase.java: ## @@ -207,6 +212,29 @@ private List

[GitHub] [jackrabbit-oak] nit0906 commented on a diff in pull request #1123: OAK-10429: Implementation for IncrementalFFS

2023-09-20 Thread via GitHub
nit0906 commented on code in PR #1123: URL: https://github.com/apache/jackrabbit-oak/pull/1123#discussion_r1331256931 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/flatfile/FlatFileStore.java: ## @@ -54,8 +57,18 @@ public

[GitHub] [jackrabbit-oak] fabriziofortino commented on a diff in pull request #1127: OAK-10449: bump zookeeper to v3.4.14

2023-09-20 Thread via GitHub
fabriziofortino commented on code in PR #1127: URL: https://github.com/apache/jackrabbit-oak/pull/1127#discussion_r1331844469 ## oak-solr-osgi/pom.xml: ## @@ -178,6 +178,12 @@ commons-math3 runtime + +org.apache.yetus +

[GitHub] [jackrabbit-oak] nfsantos commented on a diff in pull request #1127: OAK-10449: bump zookeeper to v3.4.14

2023-09-20 Thread via GitHub
nfsantos commented on code in PR #1127: URL: https://github.com/apache/jackrabbit-oak/pull/1127#discussion_r1331839958 ## oak-solr-osgi/pom.xml: ## @@ -178,6 +178,12 @@ commons-math3 runtime + +org.apache.yetus +

[GitHub] [jackrabbit-oak] fabriziofortino opened a new pull request, #1132: OAK-10456: log index name when bulk update fails

2023-09-25 Thread via GitHub
fabriziofortino opened a new pull request, #1132: URL: https://github.com/apache/jackrabbit-oak/pull/1132 (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

[GitHub] [jackrabbit-oak] reschke opened a new pull request, #1134: OAK-10461: oak-search-elastic does not build under Java 17 - disable Felix SCR plugin

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

[GitHub] [jackrabbit-oak] reschke merged pull request #1134: OAK-10461: oak-search-elastic does not build under Java 17 - disable Felix SCR plugin

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

[GitHub] [jackrabbit-oak] sonarcloud[bot] commented on pull request #1134: OAK-10461: oak-search-elastic does not build under Java 17 - disable Felix SCR plugin

2023-09-28 Thread via GitHub
sonarcloud[bot] commented on PR #1134: URL: https://github.com/apache/jackrabbit-oak/pull/1134#issuecomment-1739588561 Kudos, SonarCloud Quality Gate passed! [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png

[GitHub] [jackrabbit-oak] nit0906 merged pull request #1133: OAK-10458 - Set LZ4 as the default compression algorithm for the indexing job

2023-09-29 Thread via GitHub
nit0906 merged PR #1133: URL: https://github.com/apache/jackrabbit-oak/pull/1133 -- 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:

[GitHub] [jackrabbit-oak] tihom88 opened a new pull request, #1135: OAK-10463: Retrieve flatFileStore for indexing from baseFlatFileStore

2023-09-29 Thread via GitHub
tihom88 opened a new pull request, #1135: URL: https://github.com/apache/jackrabbit-oak/pull/1135 (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

[PR] JCRVLT-721 check for rep:AuthorizableFolder nodetype before principal policy search [jackrabbit-filevault]

2023-10-02 Thread via GitHub
adamcin opened a new pull request, #314: URL: https://github.com/apache/jackrabbit-filevault/pull/314 Replaces the user/group creation test for determining the usersPath and groupsPath config values with a condition based only on the nodetype for the node being removed. -- This is an

[PR] OAK-10458 - Set LZ4 as the default compression algorithm for the indexing job. [jackrabbit-oak]

2023-10-02 Thread via GitHub
nfsantos opened a new pull request, #1136: URL: https://github.com/apache/jackrabbit-oak/pull/1136 (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] JCRVLT-721 check for rep:AuthorizableFolder nodetype before principal policy search [jackrabbit-filevault]

2023-10-03 Thread via GitHub
kwin commented on code in PR #314: URL: https://github.com/apache/jackrabbit-filevault/pull/314#discussion_r1343669589 ## vault-core/src/main/java/org/apache/jackrabbit/vault/fs/spi/impl/jcr20/JackrabbitACLManagement.java: ## @@ -202,18 +182,22 @@ private static boolean

Re: [PR] OAK-10458 - Set LZ4 as the default compression algorithm for the indexing job. [jackrabbit-oak]

2023-10-02 Thread via GitHub
nit0906 commented on code in PR #1136: URL: https://github.com/apache/jackrabbit-oak/pull/1136#discussion_r1343534093 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/flatfile/FlatFileNodeStoreBuilder.java: ## @@ -130,7 +131,7 @@ public class

Re: [PR] JCRVLT-721 check for rep:AuthorizableFolder nodetype before principal policy search [jackrabbit-filevault]

2023-10-03 Thread via GitHub
kwin commented on PR #314: URL: https://github.com/apache/jackrabbit-filevault/pull/314#issuecomment-1744410124 Looks good, is it possible to have an IT verifying that this doesn't require particular permissions (apart from read) to prevent regressions here? -- This is an automated

[GitHub] [jackrabbit-oak] fabriziofortino merged pull request #1131: OAK-10450: bump testcontainers

2023-09-27 Thread via GitHub
fabriziofortino merged PR #1131: URL: https://github.com/apache/jackrabbit-oak/pull/1131 -- 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:

[GitHub] [jackrabbit-oak] tihom88 merged pull request #1123: OAK-10429: Implementation for IncrementalFFS

2023-09-25 Thread via GitHub
tihom88 merged PR #1123: URL: https://github.com/apache/jackrabbit-oak/pull/1123 -- 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:

[GitHub] [jackrabbit-oak] nfsantos opened a new pull request, #1133: OAK-10458 - Set LZ4 as the default compression algorithm for the indexing job

2023-09-27 Thread via GitHub
nfsantos opened a new pull request, #1133: URL: https://github.com/apache/jackrabbit-oak/pull/1133 (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

[GitHub] [jackrabbit-oak] fabriziofortino merged pull request #1132: OAK-10456: log index name when bulk update fails

2023-09-26 Thread via GitHub
fabriziofortino merged PR #1132: URL: https://github.com/apache/jackrabbit-oak/pull/1132 -- 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] JCRVLT-721 check for rep:AuthorizableFolder nodetype before principal policy search [jackrabbit-filevault]

2023-10-03 Thread via GitHub
adamcin commented on PR #314: URL: https://github.com/apache/jackrabbit-filevault/pull/314#issuecomment-1745274386 > Looks good, is it possible to have an IT verifying that this doesn't require particular permissions (apart from read) to prevent regressions here? I've added an IT,

Re: [PR] OAK-8646 : added metrics for unmergedbranch commits & improved junit coverage [jackrabbit-oak]

2023-10-03 Thread via GitHub
sonarcloud[bot] commented on PR #: URL: https://github.com/apache/jackrabbit-oak/pull/#issuecomment-1745344997 Kudos, SonarCloud Quality Gate passed! [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png

Re: [PR] OAK-10424 add support for INSECURE RESULT SIZE and INSECURE FACETS query options and rep:insecureQueryOptions privilege [jackrabbit-oak]

2023-10-03 Thread via GitHub
adamcin commented on PR #1094: URL: https://github.com/apache/jackrabbit-oak/pull/1094#issuecomment-1745396023 @anchela That's unfortunate. I guess the issue is with a new PrivilegeBits constant not being able to reflect the value of `next` in an existing privilege tree that is being

Re: [PR] OAK-8646 : added metrics for unmergedbranch commits & improved junit coverage [jackrabbit-oak]

2023-10-03 Thread via GitHub
rishabhdaim commented on PR #: URL: https://github.com/apache/jackrabbit-oak/pull/#issuecomment-1745130549 > > SonarCloud Quality Gate failed.Quality Gate failed > > maybe we can try to fix that... Fixed in

Re: [PR] JCRVLT-721 check for rep:AuthorizableFolder nodetype before principal policy search [jackrabbit-filevault]

2023-10-04 Thread via GitHub
kwin merged PR #314: URL: https://github.com/apache/jackrabbit-filevault/pull/314 -- 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-8646 : added metrics for unmergedbranch commits & improved junit coverage [jackrabbit-oak]

2023-10-04 Thread via GitHub
rishabhdaim merged PR #: URL: https://github.com/apache/jackrabbit-oak/pull/ -- 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-10451 UserPrincipalProvider may cause many conflicts when under load [jackrabbit-oak]

2023-10-04 Thread via GitHub
anchela commented on code in PR #1128: URL: https://github.com/apache/jackrabbit-oak/pull/1128#discussion_r1346059752 ## oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/PrincipalCacheConflictHandler.java: ## @@ -0,0 +1,106 @@ +/* + * Licensed to the Apache

Re: [PR] OAK-10458 - Set LZ4 as the default compression algorithm for the indexing job. [jackrabbit-oak]

2023-10-03 Thread via GitHub
reschke commented on code in PR #1136: URL: https://github.com/apache/jackrabbit-oak/pull/1136#discussion_r1343710733 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/flatfile/FlatFileNodeStoreBuilder.java: ## @@ -130,7 +131,7 @@ public class

Re: [PR] OAK-8646 : Clean up changes from orphaned branch commits [jackrabbit-oak]

2023-10-03 Thread via GitHub
stefan-egli commented on PR #1090: URL: https://github.com/apache/jackrabbit-oak/pull/1090#issuecomment-1744930719 Closed in favor of https://github.com/apache/jackrabbit-oak/pull/ /cc @rishabhdaim -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] OAK-8646 : Clean up changes from orphaned branch commits [jackrabbit-oak]

2023-10-03 Thread via GitHub
stefan-egli closed pull request #1090: OAK-8646 : Clean up changes from orphaned branch commits URL: https://github.com/apache/jackrabbit-oak/pull/1090 -- 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-8646 : added metrics for unmergedbranch commits & improved junit coverage [jackrabbit-oak]

2023-10-03 Thread via GitHub
stefan-egli commented on PR #: URL: https://github.com/apache/jackrabbit-oak/pull/#issuecomment-1744996521 > SonarCloud Quality Gate failed.Quality Gate failed maybe we can try to fix that... -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [jackrabbit-oak] fabriziofortino opened a new pull request, #1127: OAK-10449: bump zookeeper to v3.6.2 (aligned with solrj version)

2023-09-19 Thread via GitHub
fabriziofortino opened a new pull request, #1127: URL: https://github.com/apache/jackrabbit-oak/pull/1127 Update is needed because there are 2 major vulnerabilities in 3.4.10 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [jackrabbit-oak] tihom88 commented on a diff in pull request #1123: OAK-10429: Implementation for IncrementalFFS

2023-09-19 Thread via GitHub
tihom88 commented on code in PR #1123: URL: https://github.com/apache/jackrabbit-oak/pull/1123#discussion_r1329681323 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/flatfile/TraverseWithSortStrategy.java: ## @@ -92,17 +93,22 @@ class

[GitHub] [jackrabbit-oak] mbaedke opened a new pull request, #1126: OAK-10377: Checked-out state of non-versionable nodes is not handled …

2023-09-19 Thread via GitHub
mbaedke opened a new pull request, #1126: URL: https://github.com/apache/jackrabbit-oak/pull/1126 …correctly Added versionability check, test cases. -- 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

[GitHub] [jackrabbit-oak] mbaedke closed pull request #1051: OAK-10377: Checked-out state of non-versionable nodes is not handled …

2023-09-19 Thread via GitHub
mbaedke closed pull request #1051: OAK-10377: Checked-out state of non-versionable nodes is not handled … URL: https://github.com/apache/jackrabbit-oak/pull/1051 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [jackrabbit-oak] mbaedke commented on pull request #1126: OAK-10377: Checked-out state of non-versionable nodes is not handled …

2023-09-19 Thread via GitHub
mbaedke commented on PR #1126: URL: https://github.com/apache/jackrabbit-oak/pull/1126#issuecomment-1725335651 @mreutegg, I hope this addresses your change requests. I messed up the old PR#1051 by merging trunk with rebase. -- This is an automated message from the Apache Git Service. To

[GitHub] [jackrabbit-oak] thomasmueller commented on a diff in pull request #1127: OAK-10449: bump zookeeper to v3.4.14

2023-09-20 Thread via GitHub
thomasmueller commented on code in PR #1127: URL: https://github.com/apache/jackrabbit-oak/pull/1127#discussion_r1332497062 ## oak-solr-osgi/pom.xml: ## @@ -48,6 +48,7 @@ org.apache.hadoop.*;resolution:=optional,

[GitHub] [jackrabbit-oak] tihom88 commented on a diff in pull request #1123: OAK-10429: Implementation for IncrementalFFS

2023-09-25 Thread via GitHub
tihom88 commented on code in PR #1123: URL: https://github.com/apache/jackrabbit-oak/pull/1123#discussion_r1335501707 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/flatfile/FlatFileStore.java: ## @@ -54,8 +57,18 @@ public

[GitHub] [jackrabbit-oak] dulceanu closed pull request #773: OAK-3349 Partial Compaction

2023-09-22 Thread via GitHub
dulceanu closed pull request #773: OAK-3349 Partial Compaction URL: https://github.com/apache/jackrabbit-oak/pull/773 -- 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

[GitHub] [jackrabbit-oak] dulceanu commented on pull request #1043: OAK-9922: Parallel Compaction

2023-09-22 Thread via GitHub
dulceanu commented on PR #1043: URL: https://github.com/apache/jackrabbit-oak/pull/1043#issuecomment-1731397030 @reschke, given that the test failures for oak-segment-azure settled now, could you please review this? -- This is an automated message from the Apache Git Service. To respond

[GitHub] [jackrabbit-oak] sonarcloud[bot] commented on pull request #1043: OAK-9922: Parallel Compaction

2023-09-22 Thread via GitHub
sonarcloud[bot] commented on PR #1043: URL: https://github.com/apache/jackrabbit-oak/pull/1043#issuecomment-1731523583 SonarCloud Quality Gate failed. [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png

[GitHub] [jackrabbit-oak] dulceanu closed pull request #710: OAK-9949 Offline Tail Compaction

2023-09-22 Thread via GitHub
dulceanu closed pull request #710: OAK-9949 Offline Tail Compaction URL: https://github.com/apache/jackrabbit-oak/pull/710 -- 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

[GitHub] [jackrabbit-oak] fabriziofortino commented on a diff in pull request #1127: OAK-10449: bump zookeeper to v3.4.14

2023-09-21 Thread via GitHub
fabriziofortino commented on code in PR #1127: URL: https://github.com/apache/jackrabbit-oak/pull/1127#discussion_r1332561352 ## oak-solr-osgi/pom.xml: ## @@ -178,6 +178,12 @@ commons-math3 runtime + +org.apache.yetus +

[GitHub] [jackrabbit-oak] fabriziofortino commented on a diff in pull request #1127: OAK-10449: bump zookeeper to v3.4.14

2023-09-21 Thread via GitHub
fabriziofortino commented on code in PR #1127: URL: https://github.com/apache/jackrabbit-oak/pull/1127#discussion_r1332562798 ## oak-solr-osgi/pom.xml: ## @@ -48,6 +48,7 @@ org.apache.hadoop.*;resolution:=optional,

[GitHub] [jackrabbit-oak] fabriziofortino merged pull request #1127: OAK-10449: bump zookeeper to v3.4.14

2023-09-21 Thread via GitHub
fabriziofortino merged PR #1127: URL: https://github.com/apache/jackrabbit-oak/pull/1127 -- 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:

[GitHub] [jackrabbit-oak] nfsantos commented on a diff in pull request #1123: OAK-10429: Implementation for IncrementalFFS

2023-09-25 Thread via GitHub
nfsantos commented on code in PR #1123: URL: https://github.com/apache/jackrabbit-oak/pull/1123#discussion_r1335615576 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/indexstore/IndexStoreMetadataOperatorImpl.java: ## @@ -0,0 +1,70 @@ +/* + *

[GitHub] [jackrabbit-oak] fabriziofortino opened a new pull request, #1131: OAK-10450: bump testcontainers

2023-09-25 Thread via GitHub
fabriziofortino opened a new pull request, #1131: URL: https://github.com/apache/jackrabbit-oak/pull/1131 (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

[GitHub] [jackrabbit-oak] fabriziofortino commented on a diff in pull request #1123: OAK-10429: Implementation for IncrementalFFS

2023-09-25 Thread via GitHub
fabriziofortino commented on code in PR #1123: URL: https://github.com/apache/jackrabbit-oak/pull/1123#discussion_r1335595549 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/incrementalstore/MergeIncrementalFlatFileStore.java: ## @@ -0,0 +1,220

[GitHub] [jackrabbit-oak] fabriziofortino merged pull request #1129: OAK-10452 - Mongo query that downloads ancestors of base path for regex filtering is doing a column scan

2023-09-22 Thread via GitHub
fabriziofortino merged PR #1129: URL: https://github.com/apache/jackrabbit-oak/pull/1129 -- 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-10453 - Pipelined strategy: enforce size limit on memory taken by objects in the queue between download and transform thread [jackrabbit-oak]

2023-10-05 Thread via GitHub
tihom88 merged PR #1130: URL: https://github.com/apache/jackrabbit-oak/pull/1130 -- 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-10465 : added io.netty netty-transport-native-unix-common embedde… [jackrabbit-oak]

2023-10-05 Thread via GitHub
rishabhdaim opened a new pull request, #1137: URL: https://github.com/apache/jackrabbit-oak/pull/1137 …d dependency for oak-segment-tar -- 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] OAK-10458 - Set LZ4 as the default compression algorithm for the indexing job. [jackrabbit-oak]

2023-10-05 Thread via GitHub
nfsantos commented on code in PR #1136: URL: https://github.com/apache/jackrabbit-oak/pull/1136#discussion_r1347125441 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/indexstore/IndexStoreUtils.java: ## @@ -39,6 +40,23 @@ public class

Re: [PR] OAK-10463: Retrieve flatFileStore for indexing from baseFlatFileStore [jackrabbit-oak]

2023-10-04 Thread via GitHub
amit-jain commented on code in PR #1135: URL: https://github.com/apache/jackrabbit-oak/pull/1135#discussion_r1346787350 ## oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/sort/ExternalSort.java: ## @@ -410,11 +536,19 @@ public static File sortAndSave(List tmplist,

Re: [PR] OAK-10458 - Set LZ4 as the default compression algorithm for the indexing job. [jackrabbit-oak]

2023-10-05 Thread via GitHub
steffenvan commented on code in PR #1136: URL: https://github.com/apache/jackrabbit-oak/pull/1136#discussion_r1347106722 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/indexstore/IndexStoreUtils.java: ## @@ -39,6 +40,23 @@ public class

Re: [PR] OAK-10458 - Set LZ4 as the default compression algorithm for the indexing job. [jackrabbit-oak]

2023-10-05 Thread via GitHub
steffenvan commented on code in PR #1136: URL: https://github.com/apache/jackrabbit-oak/pull/1136#discussion_r1347106722 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/indexstore/IndexStoreUtils.java: ## @@ -39,6 +40,23 @@ public class

Re: [PR] OAK-10458 - Set LZ4 as the default compression algorithm for the indexing job. [jackrabbit-oak]

2023-10-05 Thread via GitHub
nfsantos commented on code in PR #1136: URL: https://github.com/apache/jackrabbit-oak/pull/1136#discussion_r1347121152 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/indexstore/IndexStoreUtils.java: ## @@ -39,6 +40,23 @@ public class

Re: [PR] OAK-10465 : added io.netty netty-transport-native-unix-common embedded dependency for oak-segment-tar [jackrabbit-oak]

2023-10-05 Thread via GitHub
reschke merged PR #1137: URL: https://github.com/apache/jackrabbit-oak/pull/1137 -- 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-10463: Retrieve flatFileStore for indexing from baseFlatFileStore [jackrabbit-oak]

2023-10-05 Thread via GitHub
tihom88 commented on code in PR #1135: URL: https://github.com/apache/jackrabbit-oak/pull/1135#discussion_r1347019114 ## oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/sort/ExternalSort.java: ## @@ -410,11 +536,19 @@ public static File sortAndSave(List tmplist,

Re: [PR] OAK-10458 - Set LZ4 as the default compression algorithm for the indexing job. [jackrabbit-oak]

2023-10-05 Thread via GitHub
steffenvan commented on code in PR #1136: URL: https://github.com/apache/jackrabbit-oak/pull/1136#discussion_r1347108455 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/indexstore/IndexStoreUtils.java: ## @@ -39,6 +40,23 @@ public class

Re: [PR] OAK-10458 - Set LZ4 as the default compression algorithm for the indexing job. [jackrabbit-oak]

2023-10-05 Thread via GitHub
steffenvan commented on code in PR #1136: URL: https://github.com/apache/jackrabbit-oak/pull/1136#discussion_r1347108455 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/indexstore/IndexStoreUtils.java: ## @@ -39,6 +40,23 @@ public class

Re: [PR] OAK-9922: Parallel Compaction [jackrabbit-oak]

2023-10-05 Thread via GitHub
dulceanu merged PR #1043: URL: https://github.com/apache/jackrabbit-oak/pull/1043 -- 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-10463: Retrieve flatFileStore for indexing from baseFlatFileStore [jackrabbit-oak]

2023-10-04 Thread via GitHub
amit-jain commented on code in PR #1135: URL: https://github.com/apache/jackrabbit-oak/pull/1135#discussion_r1346787350 ## oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/sort/ExternalSort.java: ## @@ -410,11 +536,19 @@ public static File sortAndSave(List tmplist,

Re: [PR] OAK-10458 - Set LZ4 as the default compression algorithm for the indexing job. [jackrabbit-oak]

2023-10-05 Thread via GitHub
nfsantos commented on code in PR #1136: URL: https://github.com/apache/jackrabbit-oak/pull/1136#discussion_r1347062712 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/flatfile/FlatFileNodeStoreBuilder.java: ## @@ -130,7 +131,7 @@ public class

Re: [PR] OAK-10458 - Set LZ4 as the default compression algorithm for the indexing job. [jackrabbit-oak]

2023-10-05 Thread via GitHub
nfsantos commented on code in PR #1136: URL: https://github.com/apache/jackrabbit-oak/pull/1136#discussion_r1347061053 ## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/flatfile/FlatFileNodeStoreBuilder.java: ## @@ -130,7 +131,7 @@ public class

Re: [PR] OAK-10453 - Pipelined strategy: enforce size limit on memory taken by objects in the queue between download and transform thread [jackrabbit-oak]

2023-10-05 Thread via GitHub
nfsantos commented on PR #1130: URL: https://github.com/apache/jackrabbit-oak/pull/1130#issuecomment-1748183653 > In this approach we are adding some processing to be done by mongo download thread which is the main bottleneck. Would it make sense to do these transformations on download

Re: [PR] OAK-10463: Retrieve flatFileStore for indexing from baseFlatFileStore [jackrabbit-oak]

2023-10-05 Thread via GitHub
tihom88 commented on code in PR #1135: URL: https://github.com/apache/jackrabbit-oak/pull/1135#discussion_r1347035822 ## oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/sort/ExternalSort.java: ## @@ -460,11 +602,21 @@ public static File sortAndSave(List tmplist,

Re: [PR] OAK-10463: Retrieve flatFileStore for indexing from baseFlatFileStore [jackrabbit-oak]

2023-10-05 Thread via GitHub
amit-jain commented on code in PR #1135: URL: https://github.com/apache/jackrabbit-oak/pull/1135#discussion_r1347067310 ## oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/sort/ExternalSort.java: ## @@ -460,11 +602,21 @@ public static File sortAndSave(List tmplist,

Re: [PR] OAK-10464: Use Testcontainers instead of com.arakelian:docker-junit-rule [jackrabbit-oak]

2023-10-10 Thread via GitHub
reschke commented on code in PR #1143: URL: https://github.com/apache/jackrabbit-oak/pull/1143#discussion_r1352092119 ## oak-blob-cloud-azure/pom.xml: ## @@ -191,10 +191,10 @@ mockito-core test - -com.arakelian -

Re: [PR] OAK-10474: Release Oak 1.58.0 [jackrabbit-oak]

2023-10-10 Thread via GitHub
reschke merged PR #1145: URL: https://github.com/apache/jackrabbit-oak/pull/1145 -- 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-10464: Use Testcontainers instead of com.arakelian:docker-junit-rule [jackrabbit-oak]

2023-10-10 Thread via GitHub
reschke commented on code in PR #1143: URL: https://github.com/apache/jackrabbit-oak/pull/1143#discussion_r1352503502 ## oak-segment-azure/src/test/resources/logback-test.xml: ## @@ -21,11 +21,19 @@ %d{-MM-dd HH:mm:ss.SSS} [%thread] %-5level %logger{36} -

Re: [PR] OAK-10475 - Expose the Mongo client connection in MongoDocumentNodeStoreBuilderBase [jackrabbit-oak]

2023-10-10 Thread via GitHub
nfsantos commented on code in PR #1142: URL: https://github.com/apache/jackrabbit-oak/pull/1142#discussion_r1351850691 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/mongo/MongoDocumentNodeStoreBuilderBase.java: ## @@ -212,6 +213,15 @@

Re: [PR] OAK-10464: Use Testcontainers instead of com.arakelian:docker-junit-rule [jackrabbit-oak]

2023-10-10 Thread via GitHub
reschke commented on code in PR #1143: URL: https://github.com/apache/jackrabbit-oak/pull/1143#discussion_r1352117183 ## oak-segment-azure/src/test/resources/logback-test.xml: ## @@ -21,11 +21,19 @@ %d{-MM-dd HH:mm:ss.SSS} [%thread] %-5level %logger{36} -

Re: [PR] OAK-10464: Use Testcontainers instead of com.arakelian:docker-junit-rule [jackrabbit-oak]

2023-10-10 Thread via GitHub
reschke commented on code in PR #1143: URL: https://github.com/apache/jackrabbit-oak/pull/1143#discussion_r1352093896 ## oak-blob-cloud-azure/src/test/java/org/apache/jackrabbit/oak/blob/cloud/azure/blobstorage/AzuriteDockerRule.java: ## @@ -16,55 +16,88 @@ */ package

Re: [PR] OAK-10464: Use Testcontainers instead of com.arakelian:docker-junit-rule [jackrabbit-oak]

2023-10-10 Thread via GitHub
t-rana commented on code in PR #1143: URL: https://github.com/apache/jackrabbit-oak/pull/1143#discussion_r1352111852 ## oak-segment-azure/src/test/resources/logback-test.xml: ## @@ -21,11 +21,19 @@ %d{-MM-dd HH:mm:ss.SSS} [%thread] %-5level %logger{36} -

Re: [PR] OAK-10464: Use Testcontainers instead of com.arakelian:docker-junit-rule [jackrabbit-oak]

2023-10-10 Thread via GitHub
t-rana commented on code in PR #1143: URL: https://github.com/apache/jackrabbit-oak/pull/1143#discussion_r1352125451 ## oak-lucene/pom.xml: ## @@ -471,5 +471,11 @@ 1.19.0 test + Review Comment: oak lucene uses the MongoConnectionFactory in IT(s)

Re: [PR] OAK-10464: Use Testcontainers instead of com.arakelian:docker-junit-rule [jackrabbit-oak]

2023-10-10 Thread via GitHub
reschke commented on code in PR #1143: URL: https://github.com/apache/jackrabbit-oak/pull/1143#discussion_r1352130678 ## oak-lucene/pom.xml: ## @@ -471,5 +471,11 @@ 1.19.0 test + Review Comment: But isn't that automatically inherited? -- This is

Re: [PR] OAK-10464: Use Testcontainers instead of com.arakelian:docker-junit-rule [jackrabbit-oak]

2023-10-10 Thread via GitHub
t-rana commented on code in PR #1143: URL: https://github.com/apache/jackrabbit-oak/pull/1143#discussion_r1352125451 ## oak-lucene/pom.xml: ## @@ -471,5 +471,11 @@ 1.19.0 test + Review Comment: oak-lucene uses the MongoConnectionFactory in IT(s)

Re: [PR] OAK-6760: Convert oak-blob-cloud to OSGi R6 annotations [jackrabbit-oak]

2023-10-10 Thread via GitHub
kwin commented on code in PR #1144: URL: https://github.com/apache/jackrabbit-oak/pull/1144#discussion_r1352423985 ## oak-blob-cloud/src/main/java/org/apache/jackrabbit/oak/blob/cloud/s3/S3DataStoreService.java: ## @@ -19,10 +19,23 @@ package

Re: [PR] OAK-6760: Convert oak-blob-cloud to OSGi R6 annotations [jackrabbit-oak]

2023-10-10 Thread via GitHub
reschke commented on code in PR #1144: URL: https://github.com/apache/jackrabbit-oak/pull/1144#discussion_r1352434938 ## oak-blob-cloud/src/main/java/org/apache/jackrabbit/oak/blob/cloud/s3/S3DataStoreService.java: ## @@ -19,10 +19,23 @@ package

Re: [PR] OAK-10475 - Expose the Mongo client connection in MongoDocumentNodeStoreBuilderBase [jackrabbit-oak]

2023-10-10 Thread via GitHub
thomasmueller merged PR #1142: URL: https://github.com/apache/jackrabbit-oak/pull/1142 -- 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-10464: Use Testcontainers instead of com.arakelian:docker-junit-rule [jackrabbit-oak]

2023-10-10 Thread via GitHub
t-rana commented on code in PR #1143: URL: https://github.com/apache/jackrabbit-oak/pull/1143#discussion_r1352268966 ## oak-segment-azure/src/test/resources/logback-test.xml: ## @@ -21,11 +21,19 @@ %d{-MM-dd HH:mm:ss.SSS} [%thread] %-5level %logger{36} -

[PR] OAK-10464: Use Testcontainers instead of com.arakelian:docker-junit-rule [jackrabbit-oak]

2023-10-10 Thread via GitHub
t-rana opened a new pull request, #1143: URL: https://github.com/apache/jackrabbit-oak/pull/1143 (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-6760: Convert oak-blob-cloud to OSGi R6 annotations [jackrabbit-oak]

2023-10-10 Thread via GitHub
sonarcloud[bot] commented on PR #1144: URL: https://github.com/apache/jackrabbit-oak/pull/1144#issuecomment-1755239775 SonarCloud Quality Gate failed. [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png

Re: [PR] OAK-10464: Use Testcontainers instead of com.arakelian:docker-junit-rule [jackrabbit-oak]

2023-10-10 Thread via GitHub
reschke commented on code in PR #1143: URL: https://github.com/apache/jackrabbit-oak/pull/1143#discussion_r1352344860 ## oak-lucene/pom.xml: ## @@ -471,5 +471,11 @@ 1.19.0 test + Review Comment: I was/am confused because I would have assumed that

Re: [PR] OAK-10464: Use Testcontainers instead of com.arakelian:docker-junit-rule [jackrabbit-oak]

2023-10-10 Thread via GitHub
t-rana commented on code in PR #1143: URL: https://github.com/apache/jackrabbit-oak/pull/1143#discussion_r1352270547 ## oak-blob-cloud-azure/pom.xml: ## @@ -191,10 +191,10 @@ mockito-core test - -com.arakelian -

[PR] OAK-10474: Release Oak 1.58.0 [jackrabbit-oak]

2023-10-10 Thread via GitHub
reschke opened a new pull request, #1145: URL: https://github.com/apache/jackrabbit-oak/pull/1145 (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-10464: Use Testcontainers instead of com.arakelian:docker-junit-rule [jackrabbit-oak]

2023-10-10 Thread via GitHub
t-rana commented on code in PR #1143: URL: https://github.com/apache/jackrabbit-oak/pull/1143#discussion_r1352342828 ## oak-lucene/pom.xml: ## @@ -471,5 +471,11 @@ 1.19.0 test + Review Comment: No, testcontainers dependency is defined under

Re: [PR] OAK-10472 - Move ApproximateCounter from oak-core to a place where it… [jackrabbit-oak]

2023-10-09 Thread via GitHub
sonarcloud[bot] commented on PR #1139: URL: https://github.com/apache/jackrabbit-oak/pull/1139#issuecomment-1752830777 SonarCloud Quality Gate failed. [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png

Re: [PR] OAK-10472 - Move ApproximateCounter from oak-core to a place where it… [jackrabbit-oak]

2023-10-09 Thread via GitHub
reschke commented on PR #1139: URL: https://github.com/apache/jackrabbit-oak/pull/1139#issuecomment-1752781513 Overtaken by events; see https://github.com/apache/jackrabbit-oak/commit/c747535b9274403f4d56a15ebf25aadd109700d0 - thanks @dulceanu for the quick feedback. -- This is an

[PR] OAK-10462: o.a.j.o.plugins.version.VersionEditor#propertyAdded() may … [jackrabbit-oak]

2023-10-09 Thread via GitHub
mbaedke opened a new pull request, #1141: URL: https://github.com/apache/jackrabbit-oak/pull/1141 …mistakenly assume an ongoing restore operation. Improved heuristics to identify the sentinel node of a restore operation; added unit test. -- This is an automated message from the

Re: [PR] OAK-10472 - Move ApproximateCounter from oak-core to a place where it… [jackrabbit-oak]

2023-10-09 Thread via GitHub
reschke closed pull request #1139: OAK-10472 - Move ApproximateCounter from oak-core to a place where it… URL: https://github.com/apache/jackrabbit-oak/pull/1139 -- 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-10486 : Resolution of inherited groups may terminate pre-maturely for external users [jackrabbit-oak]

2023-10-12 Thread via GitHub
mbaedke commented on PR #1147: URL: https://github.com/apache/jackrabbit-oak/pull/1147#issuecomment-1759274282 > So if I understand correctly, the problem was the early return in line 73? That would imply that swapping line 72..74 with 75..77 would fix it, too? That is also my

Re: [PR] OAK-10471: Implement ConflictHandler for UserPrincipalProvider [jackrabbit-oak]

2023-10-12 Thread via GitHub
anchela commented on code in PR #1140: URL: https://github.com/apache/jackrabbit-oak/pull/1140#discussion_r1356413938 ## oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/CacheConflictHandler.java: ## @@ -0,0 +1,95 @@ +/* + * Licensed to the Apache Software

Re: [PR] OAK-10471: Implement ConflictHandler for UserPrincipalProvider [jackrabbit-oak]

2023-10-12 Thread via GitHub
anchela commented on code in PR #1140: URL: https://github.com/apache/jackrabbit-oak/pull/1140#discussion_r1356408998 ## oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/CacheConflictHandler.java: ## @@ -0,0 +1,95 @@ +/* + * Licensed to the Apache Software

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

2023-10-12 Thread via GitHub
anchela commented on code in PR #1128: URL: https://github.com/apache/jackrabbit-oak/pull/1128#discussion_r1356466127 ## oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/PrincipalCacheCommitterThread.java: ## @@ -0,0 +1,88 @@ +/* + * Licensed to the Apache

Re: [PR] OAK-10471: Implement ConflictHandler for UserPrincipalProvider Cache [jackrabbit-oak]

2023-10-12 Thread via GitHub
anchela commented on PR #1140: URL: https://github.com/apache/jackrabbit-oak/pull/1140#issuecomment-1759298459 @mreutegg , this PR has been split from https://github.com/apache/jackrabbit-oak/pull/1128 where i requested your review. -- This is an automated message from the Apache Git

Re: [PR] OAK-10471: Implement ConflictHandler for UserPrincipalProvider Cache [jackrabbit-oak]

2023-10-12 Thread via GitHub
anchela merged PR #1140: URL: https://github.com/apache/jackrabbit-oak/pull/1140 -- 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-10490 Suggest queries return duplicate entries if prefetch is enabled [jackrabbit-oak]

2023-10-12 Thread via GitHub
thomasmueller opened a new pull request, #1148: URL: https://github.com/apache/jackrabbit-oak/pull/1148 (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

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