(bookkeeper) branch fix/github-actions-workaround deleted (was 5a64a88a6a)

2024-05-09 Thread shoothzj
This is an automated email from the ASF dual-hosted git repository. shoothzj pushed a change to branch fix/github-actions-workaround in repository https://gitbox.apache.org/repos/asf/bookkeeper.git was 5a64a88a6a Remove reference to thirdparty untrusted github actions - remove reference

(bookkeeper) branch merlimat-patch-1 deleted (was cf06328a5f)

2024-05-09 Thread shoothzj
This is an automated email from the ASF dual-hosted git repository. shoothzj pushed a change to branch merlimat-patch-1 in repository https://gitbox.apache.org/repos/asf/bookkeeper.git was cf06328a5f Enable CodeQL static code scanner The revisions that were on this branch are still

Re: [PR] build: add owasp daily build [bookkeeper]

2024-05-09 Thread via GitHub
shoothzj commented on PR #4330: URL: https://github.com/apache/bookkeeper/pull/4330#issuecomment-2102044766 @dlg99 @eolivelli @hangc0276 @merlimat @nicoloboschi @StevenLuMT @zhaijack PTAL -- This is an automated message from the Apache Git Service. To respond to the message, please log

[PR] fix: Bump apache from 29 to 32 to make maven plugins support jdk21 [bookkeeper]

2024-05-09 Thread via GitHub
shoothzj opened a new pull request, #4352: URL: https://github.com/apache/bookkeeper/pull/4352 ### Motivation Bump apache from 29 to 32, it will update some maven plugins(like maven-shade-plugin), to support jdk21 -- This is an automated message from the Apache Git Service. To

Re: [PR] Bump jackson from 2.13.4.20221013 to 2.17.1 to address CVE list [bookkeeper]

2024-05-09 Thread via GitHub
shoothzj commented on PR #4345: URL: https://github.com/apache/bookkeeper/pull/4345#issuecomment-2102076717 > we might need to switch to jdk17 for build. I will open a discuss latter maven-shade-plugin needs to update to support jdk21. `org.apache:apache` needs to be updated. #4352

Re: [PR] doc: add the missing release news [bookkeeper]

2024-05-09 Thread via GitHub
shoothzj merged PR #4341: URL: https://github.com/apache/bookkeeper/pull/4341 -- 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:

(bookkeeper) branch master updated: doc: add the missing release news (#4341)

2024-05-09 Thread shoothzj
This is an automated email from the ASF dual-hosted git repository. shoothzj pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/bookkeeper.git The following commit(s) were added to refs/heads/master by this push: new d001e3241d doc: add the missing release

(bookkeeper) branch shoothzj-patch-1 deleted (was 81e86aa6d6)

2024-05-09 Thread shoothzj
This is an automated email from the ASF dual-hosted git repository. shoothzj pushed a change to branch shoothzj-patch-1 in repository https://gitbox.apache.org/repos/asf/bookkeeper.git was 81e86aa6d6 doc: add the missing release news The revisions that were on this branch are still

Re: [PR] BP-66: support throttling for zookeeper read during rereplication [bookkeeper]

2024-05-09 Thread via GitHub
shoothzj merged PR #4256: URL: https://github.com/apache/bookkeeper/pull/4256 -- 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:

(bookkeeper) branch master updated: BP-66: support throttling for zookeeper read during rereplication (#4256)

2024-05-09 Thread shoothzj
This is an automated email from the ASF dual-hosted git repository. shoothzj pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/bookkeeper.git The following commit(s) were added to refs/heads/master by this push: new f2fdb5b5f7 BP-66: support throttling for

Re: [PR] BP-66: support throttling for zookeeper read during rereplication [bookkeeper]

2024-05-09 Thread via GitHub
shoothzj commented on PR #4256: URL: https://github.com/apache/bookkeeper/pull/4256#issuecomment-2102033945 Thanks for your first contribution, and welcome to the community -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[PR] [Improve] Change default value for CompactionMaxTimeMillis. [bookkeeper]

2024-05-09 Thread via GitHub
thetumbled opened a new pull request, #4354: URL: https://github.com/apache/bookkeeper/pull/4354 ### Motivation ### Changes (Describe: what changes you have made) > --- > In order to uphold a high standard for quality for code contributions, Apache

Re: [PR] add CompactionMaxTimeMillis in config.md [bookkeeper]

2024-05-09 Thread via GitHub
thetumbled commented on PR #4355: URL: https://github.com/apache/bookkeeper/pull/4355#issuecomment-2102360671 These conf are introduced in https://github.com/apache/bookkeeper/pull/2670, which was released in 4.14.0. So we have better add the description back into the config.md till

Re: [PR] test: fix bkHttpServiceProvider not closed in TestHttpService [bookkeeper]

2024-05-09 Thread via GitHub
shoothzj merged PR #4349: URL: https://github.com/apache/bookkeeper/pull/4349 -- 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:

(bookkeeper) branch master updated: test: fix bkHttpServiceProvider not closed in TestHttpService (#4349)

2024-05-09 Thread shoothzj
This is an automated email from the ASF dual-hosted git repository. shoothzj pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/bookkeeper.git The following commit(s) were added to refs/heads/master by this push: new d2e0e5b65c test: fix

Re: [PR] fix: Bump apache from 29 to 32 to make maven plugins support jdk21 [bookkeeper]

2024-05-09 Thread via GitHub
shoothzj closed pull request #4352: fix: Bump apache from 29 to 32 to make maven plugins support jdk21 URL: https://github.com/apache/bookkeeper/pull/4352 -- 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

Re: [PR] [improve] different bookies do not trigger gcOverreplicatedLedger at the same time [bookkeeper]

2024-05-09 Thread via GitHub
TakaHiR07 commented on PR #4314: URL: https://github.com/apache/bookkeeper/pull/4314#issuecomment-2102292256 Have fixed the test. Can you trigger CI again, thx. @shoothzj @hangc0276 @jiazhai -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] add CompactionMaxTimeMillis in config.md [bookkeeper]

2024-05-09 Thread via GitHub
shoothzj commented on code in PR #4355: URL: https://github.com/apache/bookkeeper/pull/4355#discussion_r1595237071 ## site3/website/docs/reference/config.md: ## @@ -182,7 +182,8 @@ The table below lists parameters that you can set to configure bookies. All conf |

Re: [I] BP-65: implement load balance for select bookie [bookkeeper]

2024-05-09 Thread via GitHub
eolivelli commented on issue #4247: URL: https://github.com/apache/bookkeeper/issues/4247#issuecomment-2102153080 This is a great idea +1 -- 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] fix: resource leak when JournalChannel is not fully initialized [bookkeeper]

2024-05-09 Thread via GitHub
eolivelli merged PR #4340: URL: https://github.com/apache/bookkeeper/pull/4340 -- 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:

[I] bin/bookkeeper failed on windows11 [bookkeeper]

2024-05-09 Thread via GitHub
paressuex11 opened a new issue, #4356: URL: https://github.com/apache/bookkeeper/issues/4356 **BUG REPORT** ***Describe the bug*** I tried using **git bash** to run bin/bookkeeper on windows. But it failed. error msg is like ``` $ bin/bookkeeper localbookie 1

Re: [PR] [Improve] Change default value for CompactionMaxTimeMillis. [bookkeeper]

2024-05-09 Thread via GitHub
shoothzj commented on PR #4354: URL: https://github.com/apache/bookkeeper/pull/4354#issuecomment-2102344848 I think we should add it to the history docs 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

Re: [PR] add CompactionMaxTimeMillis in config.md [bookkeeper]

2024-05-09 Thread via GitHub
thetumbled commented on code in PR #4355: URL: https://github.com/apache/bookkeeper/pull/4355#discussion_r1595239598 ## site3/website/docs/reference/config.md: ## @@ -182,7 +182,8 @@ The table below lists parameters that you can set to configure bookies. All conf |

(bookkeeper) branch master updated: fix: resource leak when JournalChannel is not fully initialized (#4340)

2024-05-09 Thread eolivelli
This is an automated email from the ASF dual-hosted git repository. eolivelli pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/bookkeeper.git The following commit(s) were added to refs/heads/master by this push: new 82a7dd3064 fix: resource leak when

Re: [PR] [Improve] Change default value for CompactionMaxTimeMillis. [bookkeeper]

2024-05-09 Thread via GitHub
shoothzj commented on PR #4354: URL: https://github.com/apache/bookkeeper/pull/4354#issuecomment-2102573301 > I think we should add it to the history docs as well. Sorry, that's comment for #4355 -- This is an automated message from the Apache Git Service. To respond to the

[PR] Bump apache from 29 to 32 to make maven plugins support jdk21 [bookkeeper]

2024-05-09 Thread via GitHub
shoothzj opened a new pull request, #4353: URL: https://github.com/apache/bookkeeper/pull/4353 (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 unsubscribe,

Re: [I] BP-65: implement load balance for select bookie [bookkeeper]

2024-05-09 Thread via GitHub
thetumbled commented on issue #4247: URL: https://github.com/apache/bookkeeper/issues/4247#issuecomment-2102144769 List some data collected from the production enviroment after we lauch this feature. A direct way to evaluate the performance of a load balancing algorithm is to look at

Re: [PR] Bump apache from 29 to 32 to make maven plugins support jdk21 [bookkeeper]

2024-05-09 Thread via GitHub
shoothzj closed pull request #4353: Bump apache from 29 to 32 to make maven plugins support jdk21 URL: https://github.com/apache/bookkeeper/pull/4353 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

Re: [PR] add CompactionMaxTimeMillis in config.md [bookkeeper]

2024-05-09 Thread via GitHub
thetumbled commented on PR #4355: URL: https://github.com/apache/bookkeeper/pull/4355#issuecomment-2102335534 PTAL, thanks. @shoothzj @wenbingshen @eolivelli @dlg99 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

(bookkeeper) branch master updated: improve: replace deprecated method usage in vertx (#4343)

2024-05-09 Thread shoothzj
This is an automated email from the ASF dual-hosted git repository. shoothzj pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/bookkeeper.git The following commit(s) were added to refs/heads/master by this push: new c87be5a79e improve: replace deprecated

Re: [PR] improve: replace deprecated method usage in vertx [bookkeeper]

2024-05-09 Thread via GitHub
shoothzj merged PR #4343: URL: https://github.com/apache/bookkeeper/pull/4343 -- 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:

(bookkeeper) branch branch-4.16 updated: Bump jackson from 2.13.4.20221013 to 2.17.1 to address CVE list (#4345)

2024-05-09 Thread shoothzj
This is an automated email from the ASF dual-hosted git repository. shoothzj pushed a commit to branch branch-4.16 in repository https://gitbox.apache.org/repos/asf/bookkeeper.git The following commit(s) were added to refs/heads/branch-4.16 by this push: new 6fb5a39e05 Bump jackson from

(bookkeeper) branch branch-4.17 updated: Bump jackson from 2.13.4.20221013 to 2.17.1 to address CVE list (#4345)

2024-05-09 Thread shoothzj
This is an automated email from the ASF dual-hosted git repository. shoothzj pushed a commit to branch branch-4.17 in repository https://gitbox.apache.org/repos/asf/bookkeeper.git The following commit(s) were added to refs/heads/branch-4.17 by this push: new 6c042fb036 Bump jackson from

[PR] add CompactionMaxTimeMillis in config.md [bookkeeper]

2024-05-09 Thread via GitHub
thetumbled opened a new pull request, #4355: URL: https://github.com/apache/bookkeeper/pull/4355 `minorCompactionMaxTimeMillis` and `majorCompactionMaxTimeMillis` is absent in `config.md`. -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] build: add owasp daily build [bookkeeper]

2024-05-09 Thread via GitHub
eolivelli merged PR #4330: URL: https://github.com/apache/bookkeeper/pull/4330 -- 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:

(bookkeeper) branch master updated: build: add owasp daily build (#4330)

2024-05-09 Thread eolivelli
This is an automated email from the ASF dual-hosted git repository. eolivelli pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/bookkeeper.git The following commit(s) were added to refs/heads/master by this push: new f030b65185 build: add owasp daily build

Re: [PR] Bump jackson from 2.13.4.20221013 to 2.17.1 to address CVE list [bookkeeper]

2024-05-09 Thread via GitHub
eolivelli merged PR #4345: URL: https://github.com/apache/bookkeeper/pull/4345 -- 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:

(bookkeeper) branch master updated: Bump jackson from 2.13.4.20221013 to 2.17.1 to address CVE list (#4345)

2024-05-09 Thread eolivelli
This is an automated email from the ASF dual-hosted git repository. eolivelli pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/bookkeeper.git The following commit(s) were added to refs/heads/master by this push: new c2df54d099 Bump jackson from

Re: [I] bin/bookkeeper failed on windows11 [bookkeeper]

2024-05-09 Thread via GitHub
zhengxiongxie commented on issue #4356: URL: https://github.com/apache/bookkeeper/issues/4356#issuecomment-2102730613 same -- 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.