[GitHub] [jackrabbit-oak] sonarcloud[bot] commented on pull request #814: OAK-10055 Auto refreshing IndexTracker causes online reindexing to fail

2023-01-06 Thread GitBox
sonarcloud[bot] commented on PR #814: URL: https://github.com/apache/jackrabbit-oak/pull/814#issuecomment-1373531366 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] sonarcloud[bot] commented on pull request #813: OAK-10054 Improved trace level logging of JCR method calls

2023-01-06 Thread GitBox
sonarcloud[bot] commented on PR #813: URL: https://github.com/apache/jackrabbit-oak/pull/813#issuecomment-1373406970 SonarCloud Quality Gate failed. [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality

[GitHub] [jackrabbit-oak] mreutegg commented on a diff in pull request #812: OAK-10034 : created an background thread to purge unmerged branch com…

2023-01-05 Thread GitBox
mreutegg commented on code in PR #812: URL: https://github.com/apache/jackrabbit-oak/pull/812#discussion_r1062602664 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentNodeStore.java: ## @@ -3699,6 +3714,24 @@ private static Supplier

[GitHub] [jackrabbit-oak] mreutegg merged pull request #796: OAK-10031 : purge uncommitted revisions and collisions in batches on …

2023-01-05 Thread GitBox
mreutegg merged PR #796: URL: https://github.com/apache/jackrabbit-oak/pull/796 -- 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] blackat commented on pull request #811: Update Netty Project to version 4.1.86.Final to resolve CVE-2022-41915, cve-2022-24823, CVE-2021-43797, CVE-2022-41881

2023-01-05 Thread GitBox
blackat commented on PR #811: URL: https://github.com/apache/jackrabbit-oak/pull/811#issuecomment-1372029131 > > I don't have an account to open a Jira Ticket and I have followed the guide but it is not clear how to request an account, could you please provide me the email address to

[GitHub] [jackrabbit-oak] reschke commented on pull request #811: Update Netty Project to version 4.1.86.Final to resolve CVE-2022-41915, cve-2022-24823, CVE-2021-43797, CVE-2022-41881

2023-01-04 Thread GitBox
reschke commented on PR #811: URL: https://github.com/apache/jackrabbit-oak/pull/811#issuecomment-1371861743 > I don't have an account to open a Jira Ticket and I have followed the guide but it is not clear how to request an account, could you please provide me the email address to request

[GitHub] [jackrabbit-filevault] reschke merged pull request #265: JCRVLT-675: Release Filevault 3.6.8 - Release Notes

2023-01-04 Thread GitBox
reschke merged PR #265: URL: https://github.com/apache/jackrabbit-filevault/pull/265 -- 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] blackat commented on pull request #811: Update Netty Project to version 4.1.86.Final to resolve CVE-2022-41915, cve-2022-24823, CVE-2021-43797, CVE-2022-41881

2023-01-04 Thread GitBox
blackat commented on PR #811: URL: https://github.com/apache/jackrabbit-oak/pull/811#issuecomment-1371118376 > Note: changes to Oak require a JIRA ticket. Hello, I don't have an account to open a Jira Ticket and I have try to follow the guide but it is not clear how to request an

[GitHub] [jackrabbit-oak] rishabhdaim opened a new pull request, #812: OAK-10034 : created an background thread to purge unmerged branch com…

2023-01-04 Thread GitBox
rishabhdaim opened a new pull request, #812: URL: https://github.com/apache/jackrabbit-oak/pull/812 …mits and collision markers for non active clusters -- 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

[GitHub] [jackrabbit-oak] reschke commented on pull request #811: Update Netty Project to version 4.1.86.Final to resolve CVE-2022-41915, cve-2022-24823, CVE-2021-43797, CVE-2022-41881

2023-01-03 Thread GitBox
reschke commented on PR #811: URL: https://github.com/apache/jackrabbit-oak/pull/811#issuecomment-1370013766 Note: changes to Oak require a JIRA ticket. -- 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

[GitHub] [jackrabbit-oak] blackat opened a new pull request, #811: Update Netty Project to version 4.1.86.Final to resolve CVE-2022-41915, cve-2022-24823, CVE-2021-43797, CVE-2022-41881

2023-01-03 Thread GitBox
blackat opened a new pull request, #811: URL: https://github.com/apache/jackrabbit-oak/pull/811 Update Netty Project to version 4.1.86.Final to resolve https://github.com/advisories/GHSA-hh82-3pmq-7frp, https://github.com/advisories/GHSA-269q-hmxg-m83q,

[GitHub] [jackrabbit-oak] blackat closed pull request #321: Update netty to resolve CVE-2021-21295 and BDSA-2018-4022

2023-01-03 Thread GitBox
blackat closed pull request #321: Update netty to resolve CVE-2021-21295 and BDSA-2018-4022 URL: https://github.com/apache/jackrabbit-oak/pull/321 -- 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

[GitHub] [jackrabbit-oak] kwin merged pull request #810: OAK-10051 attach correct artifact

2023-01-02 Thread GitBox
kwin merged PR #810: URL: https://github.com/apache/jackrabbit-oak/pull/810 -- 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] reschke commented on pull request #810: OAK-10051 attach correct artifact

2023-01-01 Thread GitBox
reschke commented on PR #810: URL: https://github.com/apache/jackrabbit-oak/pull/810#issuecomment-1368468020 Ah, good point. -- 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.

[GitHub] [jackrabbit-oak] kwin commented on pull request #810: OAK-10051 attach correct artifact

2023-01-01 Thread GitBox
kwin commented on PR #810: URL: https://github.com/apache/jackrabbit-oak/pull/810#issuecomment-1368460685 Just execute `mvn install` on that module and look in the log which artifact is installed in the local repo. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [jackrabbit-oak] reschke commented on pull request #810: OAK-10051 attach correct artifact

2023-01-01 Thread GitBox
reschke commented on PR #810: URL: https://github.com/apache/jackrabbit-oak/pull/810#issuecomment-1368460362 Is there a simple way to test this? -- 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

[GitHub] [jackrabbit-oak] github-actions[bot] commented on pull request #110: OAK-7891: Fix unexpected test failures caused by DataStoreUtils.testPropertySetup

2022-12-27 Thread GitBox
github-actions[bot] commented on PR #110: URL: https://github.com/apache/jackrabbit-oak/pull/110#issuecomment-1366309711 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

[GitHub] [jackrabbit-oak] sonarcloud[bot] commented on pull request #810: OAK-10051 attach correct artifact

2022-12-27 Thread GitBox
sonarcloud[bot] commented on PR #810: URL: https://github.com/apache/jackrabbit-oak/pull/810#issuecomment-1365781847 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] kwin opened a new pull request, #810: OAK-10051 attach correct artifact

2022-12-27 Thread GitBox
kwin opened a new pull request, #810: URL: https://github.com/apache/jackrabbit-oak/pull/810 Use the one from m-assembly-p instead of the one from m-jar-p -- 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] github-actions[bot] commented on pull request #117: Return DSGC failure when 'Not all repositories have marked references available' is logged

2022-12-26 Thread GitBox
github-actions[bot] commented on PR #117: URL: https://github.com/apache/jackrabbit-oak/pull/117#issuecomment-1365548212 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

[GitHub] [jackrabbit-oak] github-actions[bot] commented on pull request #118: OAK-7951 backport for 1.6

2022-12-25 Thread GitBox
github-actions[bot] commented on PR #118: URL: https://github.com/apache/jackrabbit-oak/pull/118#issuecomment-1364805261 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

[GitHub] [jackrabbit-oak] github-actions[bot] closed pull request #131: OAK-8344 | Exposing local index directory size as a metric

2022-12-25 Thread GitBox
github-actions[bot] closed pull request #131: OAK-8344 | Exposing local index directory size as a metric URL: https://github.com/apache/jackrabbit-oak/pull/131 -- 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] github-actions[bot] commented on pull request #119: OAK-7951 backport for 1.8

2022-12-24 Thread GitBox
github-actions[bot] commented on PR #119: URL: https://github.com/apache/jackrabbit-oak/pull/119#issuecomment-1364610831 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

[GitHub] [jackrabbit-oak] github-actions[bot] closed pull request #132: OAK-7254: Indexes with excludedPaths, or includedPaths should not be …

2022-12-24 Thread GitBox
github-actions[bot] closed pull request #132: OAK-7254: Indexes with excludedPaths, or includedPaths should not be … URL: https://github.com/apache/jackrabbit-oak/pull/132 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [jackrabbit-oak] github-actions[bot] commented on pull request #120: OAK-8035 | Debug message not getting logged for index plans with simi…

2022-12-23 Thread GitBox
github-actions[bot] commented on PR #120: URL: https://github.com/apache/jackrabbit-oak/pull/120#issuecomment-1364436119 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

[GitHub] [jackrabbit-oak] github-actions[bot] closed pull request #133: Oak-8449

2022-12-23 Thread GitBox
github-actions[bot] closed pull request #133: Oak-8449 URL: https://github.com/apache/jackrabbit-oak/pull/133 -- 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,

[GitHub] [jackrabbit-oak] github-actions[bot] commented on pull request #121: Granite-24081: Index Copier Stats MBean shows stale info

2022-12-22 Thread GitBox
github-actions[bot] commented on PR #121: URL: https://github.com/apache/jackrabbit-oak/pull/121#issuecomment-1363535220 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

[GitHub] [jackrabbit-oak] github-actions[bot] closed pull request #134: OAK-8466 : Old inactive clusterIds may trigger expensive recovery

2022-12-22 Thread GitBox
github-actions[bot] closed pull request #134: OAK-8466 : Old inactive clusterIds may trigger expensive recovery URL: https://github.com/apache/jackrabbit-oak/pull/134 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [jackrabbit-oak] smiroslav commented on pull request #800: Issue/oak 10022

2022-12-22 Thread GitBox
smiroslav commented on PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#issuecomment-1362895322 SonarCloud checks have been added recently, I believe, by @nit0906 . It will gradually improve the codebase, but I do not think every check should be a blocker for a PR merge,

[GitHub] [jackrabbit-oak] reschke merged pull request #808: OAK-9800: Update Maven Plugins and Apache Parent POM to version 29

2022-12-22 Thread GitBox
reschke merged PR #808: URL: https://github.com/apache/jackrabbit-oak/pull/808 -- 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 #800: Issue/oak 10022

2022-12-22 Thread GitBox
sonarcloud[bot] commented on PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#issuecomment-1362752312 SonarCloud Quality Gate failed. [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality

[GitHub] [jackrabbit-oak] github-actions[bot] closed pull request #135: OAK-7917 | Adding the basic wirings and a test for a simple use case …

2022-12-21 Thread GitBox
github-actions[bot] closed pull request #135: OAK-7917 | Adding the basic wirings and a test for a simple use case … URL: https://github.com/apache/jackrabbit-oak/pull/135 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [jackrabbit-oak] github-actions[bot] closed pull request #137: OAK-8521 : Adding shut down hook to close, Closer which will free all

2022-12-21 Thread GitBox
github-actions[bot] closed pull request #137: OAK-8521 : Adding shut down hook to close, Closer which will free all URL: https://github.com/apache/jackrabbit-oak/pull/137 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [jackrabbit-oak] jelmini commented on pull request #800: Issue/oak 10022

2022-12-21 Thread GitBox
jelmini commented on PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#issuecomment-1362050435 This new SonarCloud check is failing the build because of not enough coverage (<80%) for the modified classes. For example Sonar considers that `AbstractRemoteSegmentArchiveWriter`

[GitHub] [jackrabbit-oak] sonarcloud[bot] commented on pull request #800: Issue/oak 10022

2022-12-21 Thread GitBox
sonarcloud[bot] commented on PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#issuecomment-1361699722 SonarCloud Quality Gate failed. [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality

[GitHub] [jackrabbit-oak] jelmini commented on a diff in pull request #800: Issue/oak 10022

2022-12-21 Thread GitBox
jelmini commented on code in PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#discussion_r1054428182 ## oak-segment-azure/src/test/java/org/apache/jackrabbit/oak/segment/azure/AzureArchiveManagerTest.java: ## @@ -400,6 +400,40 @@ public void

[GitHub] [jackrabbit-oak] jelmini commented on a diff in pull request #800: Issue/oak 10022

2022-12-21 Thread GitBox
jelmini commented on code in PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#discussion_r1054418036 ## oak-segment-azure/src/test/java/org/apache/jackrabbit/oak/segment/azure/AzureArchiveManagerTest.java: ## @@ -400,6 +400,40 @@ public void

[GitHub] [jackrabbit-oak] sonarcloud[bot] commented on pull request #800: Issue/oak 10022

2022-12-21 Thread GitBox
sonarcloud[bot] commented on PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#issuecomment-1361301883 SonarCloud Quality Gate failed. [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality

[GitHub] [jackrabbit-oak] ahanikel commented on pull request #809: OAK-10049 - Enable access to the secondary Azure blobstore service endpoint in Oak blobstore

2022-12-21 Thread GitBox
ahanikel commented on PR #809: URL: https://github.com/apache/jackrabbit-oak/pull/809#issuecomment-1361254003 @smiroslav @dulceanu Looks like I can't add any reviewers, so I'm using a good old ping. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [jackrabbit-oak] ahanikel commented on pull request #800: Issue/oak 10022

2022-12-21 Thread GitBox
ahanikel commented on PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#issuecomment-1361198772 @smiroslav @jelmini Yes, I think that's fine. -- 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] tihom88 merged pull request #795: OAK-10033:Conditions on dates use the wrong range

2022-12-20 Thread GitBox
tihom88 merged PR #795: URL: https://github.com/apache/jackrabbit-oak/pull/795 -- 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] reschke merged pull request #806: OAK-10047: oak-core: bump up logging for deprecated Guava based APIs to WARN

2022-12-20 Thread GitBox
reschke merged PR #806: URL: https://github.com/apache/jackrabbit-oak/pull/806 -- 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 #808: OAK-9800: Update Maven Plugins and Apache Parent POM to version 29

2022-12-20 Thread GitBox
sonarcloud[bot] commented on PR #808: URL: https://github.com/apache/jackrabbit-oak/pull/808#issuecomment-1360067174 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] sonarcloud[bot] commented on pull request #808: OAK-9800: Update Maven Plugins and Apache Parent POM to version 29

2022-12-20 Thread GitBox
sonarcloud[bot] commented on PR #808: URL: https://github.com/apache/jackrabbit-oak/pull/808#issuecomment-1359775074 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] reschke commented on a diff in pull request #808: OAK-9800: Update Maven Plugins and Apache Parent POM to version 29

2022-12-20 Thread GitBox
reschke commented on code in PR #808: URL: https://github.com/apache/jackrabbit-oak/pull/808#discussion_r1053527935 ## oak-parent/pom.xml: ## @@ -276,36 +269,32 @@ gmavenplus-plugin 1.13.1 + Review Comment: Indeed. -- This is an

[GitHub] [jackrabbit-filevault] stoerr commented on pull request #260: JCRVLT-672 use mavens ComparableVersion for package Version compareTo

2022-12-20 Thread GitBox
stoerr commented on PR #260: URL: https://github.com/apache/jackrabbit-filevault/pull/260#issuecomment-1359667706 Thank you as well! -- 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

[GitHub] [jackrabbit-oak] jelmini commented on pull request #800: Issue/oak 10022

2022-12-20 Thread GitBox
jelmini commented on PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#issuecomment-1359657107 > @jelmini @ahanikel how about declaring method in SegmentArchiveWriter > > ``` > public boolean isRemoteStoreWriter(); // or some other better name ;) > ``` Yep,

[GitHub] [jackrabbit-oak] kwin commented on a diff in pull request #808: OAK-9800: Update Maven Plugins and Apache Parent POM to version 29

2022-12-20 Thread GitBox
kwin commented on code in PR #808: URL: https://github.com/apache/jackrabbit-oak/pull/808#discussion_r1053498706 ## oak-parent/pom.xml: ## @@ -276,36 +269,32 @@ gmavenplus-plugin 1.13.1 + Review Comment: duplicates lines 268ff --

[GitHub] [jackrabbit-oak] reschke opened a new pull request, #808: OAK-9800: Update Maven Plugins and Apache Parent POM to version 29

2022-12-20 Thread GitBox
reschke opened a new pull request, #808: URL: https://github.com/apache/jackrabbit-oak/pull/808 (based on @kwin's work) -- 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-filevault] kwin commented on pull request #260: JCRVLT-672 use mavens ComparableVersion for package Version compareTo

2022-12-20 Thread GitBox
kwin commented on PR #260: URL: https://github.com/apache/jackrabbit-filevault/pull/260#issuecomment-1359622814 Merged a slightly modified version in https://github.com/apache/jackrabbit-filevault/commit/1442a06b288b0f90f3e78ccd5ad0291f977675bb. @stoerr Thanks for the efforts. -- This

[GitHub] [jackrabbit-filevault] kwin closed pull request #260: JCRVLT-672 use mavens ComparableVersion for package Version compareTo

2022-12-20 Thread GitBox
kwin closed pull request #260: JCRVLT-672 use mavens ComparableVersion for package Version compareTo URL: https://github.com/apache/jackrabbit-filevault/pull/260 -- 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-filevault] kwin merged pull request #264: releng: Use ASF parent 29

2022-12-20 Thread GitBox
kwin merged PR #264: URL: https://github.com/apache/jackrabbit-filevault/pull/264 -- 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] smiroslav commented on pull request #800: Issue/oak 10022

2022-12-20 Thread GitBox
smiroslav commented on PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#issuecomment-1359477688 @jelmini @ahanikel how about declaring method in SegmentArchiveWriter ``` public boolean isRemoteStoreWriter() ``` And use it in

[GitHub] [jackrabbit-oak] smiroslav commented on a diff in pull request #800: Issue/oak 10022

2022-12-20 Thread GitBox
smiroslav commented on code in PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#discussion_r1053340481 ## oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/tar/TarWriter.java: ## @@ -183,7 +183,14 @@ void flush() throws IOException {

[GitHub] [jackrabbit-oak] smiroslav commented on a diff in pull request #800: Issue/oak 10022

2022-12-20 Thread GitBox
smiroslav commented on code in PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#discussion_r1053340481 ## oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/tar/TarWriter.java: ## @@ -183,7 +183,14 @@ void flush() throws IOException {

[GitHub] [jackrabbit-oak] smiroslav commented on a diff in pull request #800: Issue/oak 10022

2022-12-20 Thread GitBox
smiroslav commented on code in PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#discussion_r1053340481 ## oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/tar/TarWriter.java: ## @@ -183,7 +183,14 @@ void flush() throws IOException {

[GitHub] [jackrabbit-oak] sonarcloud[bot] commented on pull request #807: OAK-10046: oak-it-osgi: remove unneeded workaround maven-resources-plugin

2022-12-20 Thread GitBox
sonarcloud[bot] commented on PR #807: URL: https://github.com/apache/jackrabbit-oak/pull/807#issuecomment-1359334475 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] sonarcloud[bot] commented on pull request #806: OAK-10047: oak-core: bump up logging for deprecated Guava based APIs to WARN

2022-12-20 Thread GitBox
sonarcloud[bot] commented on PR #806: URL: https://github.com/apache/jackrabbit-oak/pull/806#issuecomment-1359320891 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] reschke merged pull request #807: OAK-10046: oak-it-osgi: remove unneeded workaround maven-resources-plugin

2022-12-20 Thread GitBox
reschke merged PR #807: URL: https://github.com/apache/jackrabbit-oak/pull/807 -- 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] jelmini commented on pull request #800: Issue/oak 10022

2022-12-20 Thread GitBox
jelmini commented on PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#issuecomment-1359240904 > @smiroslav I think there's a potential problem here: tar archives are append-only, so if we write the references after each flush, we will produce a lot of waste. That isn't a

[GitHub] [jackrabbit-filevault] stoerr commented on pull request #260: JCRVLT-672 use mavens ComparableVersion for package Version compareTo

2022-12-20 Thread GitBox
stoerr commented on PR #260: URL: https://github.com/apache/jackrabbit-filevault/pull/260#issuecomment-1359177324 Ouch! I'd never ever would have found that problem in the logs.  Thank you! -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [jackrabbit-filevault] reschke merged pull request #263: JCRVLT-670: remove Weak ReferenceMap used for String caching, removing commons-collections dependency

2022-12-20 Thread GitBox
reschke merged PR #263: URL: https://github.com/apache/jackrabbit-filevault/pull/263 -- 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-filevault] kwin commented on a diff in pull request #260: JCRVLT-672 use mavens ComparableVersion for package Version compareTo

2022-12-20 Thread GitBox
kwin commented on code in PR #260: URL: https://github.com/apache/jackrabbit-filevault/pull/260#discussion_r1053166737 ## vault-core/src/main/java/org/apache/jackrabbit/vault/packaging/Version.java: ## @@ -135,44 +136,40 @@ public String[] getNormalizedSegments() { }

[GitHub] [jackrabbit-filevault] reschke closed pull request #261: JCRVLT-670: remove use String cache, getting rid of commons-collections dependency

2022-12-20 Thread GitBox
reschke closed pull request #261: JCRVLT-670: remove use String cache, getting rid of commons-collections dependency URL: https://github.com/apache/jackrabbit-filevault/pull/261 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [jackrabbit-filevault] reschke merged pull request #259: JCRVLT-673: AggregateImpl: do not collect & sort nodes that will be s…

2022-12-20 Thread GitBox
reschke merged PR #259: URL: https://github.com/apache/jackrabbit-filevault/pull/259 -- 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-filevault] reschke commented on pull request #259: JCRVLT-673: AggregateImpl: do not collect & sort nodes that will be s…

2022-12-20 Thread GitBox
reschke commented on PR #259: URL: https://github.com/apache/jackrabbit-filevault/pull/259#issuecomment-1359130803 OK - I'll go ahead then. -- 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

[GitHub] [jackrabbit-filevault] reschke commented on pull request #259: JCRVLT-673: AggregateImpl: do not collect & sort nodes that will be s…

2022-12-20 Thread GitBox
reschke commented on PR #259: URL: https://github.com/apache/jackrabbit-filevault/pull/259#issuecomment-1359020573 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.

[GitHub] [jackrabbit-oak] ahanikel commented on pull request #800: Issue/oak 10022

2022-12-20 Thread GitBox
ahanikel commented on PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#issuecomment-1359013262 @smiroslav I think there's a potential problem here: tar archives are append-only, so if we write the references after each flush, we will produce a lot of waste. That isn't a

[GitHub] [jackrabbit-filevault] kwin commented on pull request #259: JCRVLT-673: AggregateImpl: do not collect & sort nodes that will be s…

2022-12-20 Thread GitBox
kwin commented on PR #259: URL: https://github.com/apache/jackrabbit-filevault/pull/259#issuecomment-1358993165 @reschke Can you rebase to make the ASF Jenkins build succeed? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [jackrabbit-filevault] reschke commented on pull request #259: JCRVLT-673: AggregateImpl: do not collect & sort nodes that will be s…

2022-12-20 Thread GitBox
reschke commented on PR #259: URL: https://github.com/apache/jackrabbit-filevault/pull/259#issuecomment-1358985323 @kwin - this passes tests, and it has shown a good improvement in out production use case. Could you please review? -- This is an automated message from the Apache Git

[GitHub] [jackrabbit-oak] github-actions[bot] commented on pull request #122: Oak 8114

2022-12-19 Thread GitBox
github-actions[bot] commented on PR #122: URL: https://github.com/apache/jackrabbit-oak/pull/122#issuecomment-1358728169 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

[GitHub] [jackrabbit-oak] sonarcloud[bot] commented on pull request #805: OAK-10045: Upgrade gmavenplus-plugin to 1.13.1

2022-12-19 Thread GitBox
sonarcloud[bot] commented on PR #805: URL: https://github.com/apache/jackrabbit-oak/pull/805#issuecomment-1358171378 Kudos, SonarCloud Quality Gate passed! [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png

[GitHub] [jackrabbit-filevault] kwin commented on pull request #260: JCRVLT-672 use mavens ComparableVersion for package Version compareTo

2022-12-19 Thread GitBox
kwin commented on PR #260: URL: https://github.com/apache/jackrabbit-filevault/pull/260#issuecomment-1358105973 Build failed due to invalid javadoc: `[ERROR]

[GitHub] [jackrabbit-oak] reschke merged pull request #805: OAK-10045: Upgrade gmavenplus-plugin to 1.13.1

2022-12-19 Thread GitBox
reschke merged PR #805: URL: https://github.com/apache/jackrabbit-oak/pull/805 -- 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-filevault] stoerr commented on pull request #260: JCRVLT-672 use mavens ComparableVersion for package Version compareTo

2022-12-19 Thread GitBox
stoerr commented on PR #260: URL: https://github.com/apache/jackrabbit-filevault/pull/260#issuecomment-1358061722 @kwin Puzzling. The build worked after the rebase, but after the review changes it doesn't work, again. And it still shows "1 change requested" although all review comments

[GitHub] [jackrabbit-oak] jelmini commented on a diff in pull request #800: Issue/oak 10022

2022-12-19 Thread GitBox
jelmini commented on code in PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#discussion_r1052445924 ## oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/tar/TarWriter.java: ## @@ -183,7 +183,14 @@ void flush() throws IOException {

[GitHub] [jackrabbit-oak] sonarcloud[bot] commented on pull request #804: OAK-10043: Upgrade maven-versions-plugin to 2.14.1

2022-12-19 Thread GitBox
sonarcloud[bot] commented on PR #804: URL: https://github.com/apache/jackrabbit-oak/pull/804#issuecomment-1357972064 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] jelmini commented on a diff in pull request #800: Issue/oak 10022

2022-12-19 Thread GitBox
jelmini commented on code in PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#discussion_r1052435064 ## oak-segment-remote/src/main/java/org/apache/jackrabbit/oak/segment/remote/AbstractRemoteSegmentArchiveWriter.java: ## @@ -150,6 +150,12 @@ public void flush()

[GitHub] [jackrabbit-oak] jelmini commented on a diff in pull request #800: Issue/oak 10022

2022-12-19 Thread GitBox
jelmini commented on code in PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#discussion_r1052432848 ## oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/tar/TarWriter.java: ## @@ -183,7 +183,14 @@ void flush() throws IOException {

[GitHub] [jackrabbit-oak] sonarcloud[bot] commented on pull request #803: OAK-10044: Javadoc fixes for JDK 11 compat

2022-12-19 Thread GitBox
sonarcloud[bot] commented on PR #803: URL: https://github.com/apache/jackrabbit-oak/pull/803#issuecomment-1357942686 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] smiroslav commented on a diff in pull request #800: Issue/oak 10022

2022-12-19 Thread GitBox
smiroslav commented on code in PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#discussion_r1052417750 ## oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/tar/TarWriter.java: ## @@ -183,7 +183,14 @@ void flush() throws IOException {

[GitHub] [jackrabbit-oak] reschke merged pull request #804: OAK-10043: Upgrade maven-versions-plugin to 2.14.1

2022-12-19 Thread GitBox
reschke merged PR #804: URL: https://github.com/apache/jackrabbit-oak/pull/804 -- 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 #802: OAK-10042: Update checkstyle-plugin dependency to version 3.1.2

2022-12-19 Thread GitBox
sonarcloud[bot] commented on PR #802: URL: https://github.com/apache/jackrabbit-oak/pull/802#issuecomment-1357875912 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] reschke merged pull request #803: OAK-10044: Javadoc fixes for JDK 11 compat

2022-12-19 Thread GitBox
reschke merged PR #803: URL: https://github.com/apache/jackrabbit-oak/pull/803 -- 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] reschke merged pull request #802: OAK-10042: Update checkstyle-plugin dependency to version 3.1.2

2022-12-19 Thread GitBox
reschke merged PR #802: URL: https://github.com/apache/jackrabbit-oak/pull/802 -- 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 #801: OAK-10041: Upgrade maven-bundle-plugin to 5.1.8

2022-12-19 Thread GitBox
sonarcloud[bot] commented on PR #801: URL: https://github.com/apache/jackrabbit-oak/pull/801#issuecomment-1357753233 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] sonarcloud[bot] commented on pull request #800: Issue/oak 10022

2022-12-19 Thread GitBox
sonarcloud[bot] commented on PR #800: URL: https://github.com/apache/jackrabbit-oak/pull/800#issuecomment-1357722092 SonarCloud Quality Gate failed. [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality

[GitHub] [jackrabbit-filevault] stoerr commented on a diff in pull request #260: JCRVLT-672 use mavens ComparableVersion for package Version compareTo

2022-12-19 Thread GitBox
stoerr commented on code in PR #260: URL: https://github.com/apache/jackrabbit-filevault/pull/260#discussion_r1052226353 ## vault-core/src/main/java/org/apache/jackrabbit/vault/packaging/Version.java: ## @@ -135,44 +136,40 @@ public String[] getNormalizedSegments() { }

[GitHub] [jackrabbit-oak] reschke merged pull request #801: OAK-10041: Upgrade maven-bundle-plugin to 5.1.8

2022-12-19 Thread GitBox
reschke merged PR #801: URL: https://github.com/apache/jackrabbit-oak/pull/801 -- 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 #799: OAK-10040: Update animal-sniffer dependency to 1.22

2022-12-19 Thread GitBox
sonarcloud[bot] commented on PR #799: URL: https://github.com/apache/jackrabbit-oak/pull/799#issuecomment-1357639391 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] dulceanu merged pull request #790: OAK-9911 - Resource leak in ChunkedBlobStream

2022-12-19 Thread GitBox
dulceanu merged PR #790: URL: https://github.com/apache/jackrabbit-oak/pull/790 -- 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] smiroslav opened a new pull request, #800: Issue/oak 10022

2022-12-19 Thread GitBox
smiroslav opened a new pull request, #800: URL: https://github.com/apache/jackrabbit-oak/pull/800 https://issues.apache.org/jira/browse/OAK-10022 -- 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

[GitHub] [jackrabbit-oak] reschke merged pull request #799: OAK-10040: Update animal-sniffer dependency to 1.22

2022-12-19 Thread GitBox
reschke merged PR #799: URL: https://github.com/apache/jackrabbit-oak/pull/799 -- 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] reschke merged pull request #798: OAK-10039: CompositeTestSupport: improve diagnostics

2022-12-19 Thread GitBox
reschke merged PR #798: URL: https://github.com/apache/jackrabbit-oak/pull/798 -- 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] reschke opened a new pull request, #798: OAK-10039: CompositeTestSupport: improve diagnostics

2022-12-19 Thread GitBox
reschke opened a new pull request, #798: URL: https://github.com/apache/jackrabbit-oak/pull/798 (improvements provided by by @kwin in https://github.com/apache/jackrabbit-oak/pull/592) -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [jackrabbit-filevault] kwin commented on a diff in pull request #260: JCRVLT-672 use mavens ComparableVersion for package Version compareTo

2022-12-19 Thread GitBox
kwin commented on code in PR #260: URL: https://github.com/apache/jackrabbit-filevault/pull/260#discussion_r1051910062 ## vault-core/src/main/java/org/apache/jackrabbit/vault/packaging/Version.java: ## @@ -135,44 +136,40 @@ public String[] getNormalizedSegments() { }

[GitHub] [jackrabbit-oak] mreutegg commented on a diff in pull request #796: OAK-10031 : purge uncommitted revisions and collisions in batches on …

2022-12-18 Thread GitBox
mreutegg commented on code in PR #796: URL: https://github.com/apache/jackrabbit-oak/pull/796#discussion_r1051894822 ## oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/NodeDocument.java: ## @@ -563,67 +565,81 @@ public boolean

[GitHub] [jackrabbit-oak] github-actions[bot] commented on pull request #123: OAK-8138 | Switching statistical facet mode to secure in case hit cou…

2022-12-18 Thread GitBox
github-actions[bot] commented on PR #123: URL: https://github.com/apache/jackrabbit-oak/pull/123#issuecomment-1356976746 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

[GitHub] [jackrabbit-oak] github-actions[bot] commented on pull request #124: OAK-8233 use System.currentTimeMillis()

2022-12-18 Thread GitBox
github-actions[bot] commented on PR #124: URL: https://github.com/apache/jackrabbit-oak/pull/124#issuecomment-1356976713 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

[GitHub] [jackrabbit-filevault] dependabot[bot] commented on pull request #252: build(deps): bump woodstox-core from 6.1.1 to 6.4.0 in /vault-core

2022-12-18 Thread GitBox
dependabot[bot] commented on PR #252: URL: https://github.com/apache/jackrabbit-filevault/pull/252#issuecomment-1356850178 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor

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