[GitHub] [bookkeeper] codecov-commenter commented on pull request #3799: Support retry logic for auto recovery

2023-02-21 Thread via GitHub
codecov-commenter commented on PR #3799: URL: https://github.com/apache/bookkeeper/pull/3799#issuecomment-1439567423 #

[GitHub] [bookkeeper] wolfstudy opened a new pull request, #3799: Support retry logic for auto recovery

2023-02-21 Thread via GitHub
wolfstudy opened a new pull request, #3799: URL: https://github.com/apache/bookkeeper/pull/3799 Descriptions of the changes in this PR: ### Motivation In the current implementation, when the Session between Bookie and ZK expires, Auto Recovery does not support

[GitHub] [bookkeeper] codecov-commenter commented on pull request #3796: BP61: revert BP-59 to release ByteBuf using ReferenceCountUtil.release() instead of ReferenceCountUtil.safeRelease()

2023-02-21 Thread via GitHub
codecov-commenter commented on PR #3796: URL: https://github.com/apache/bookkeeper/pull/3796#issuecomment-1439441273 #

[GitHub] [bookkeeper] hangc0276 opened a new pull request, #3798: [Branch-4.14] Use ReferenceCountUtil.release() instead of ReferenceCountUtil.safeRelease()

2023-02-21 Thread via GitHub
hangc0276 opened a new pull request, #3798: URL: https://github.com/apache/bookkeeper/pull/3798 Fix #3792 for branch-4.14 -- 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] [bookkeeper] hangc0276 opened a new pull request, #3797: [Branch-4.15] Use ReferenceCountUtil.release() instead of ReferenceCountUtil.safeRelease()

2023-02-21 Thread via GitHub
hangc0276 opened a new pull request, #3797: URL: https://github.com/apache/bookkeeper/pull/3797 Related to https://github.com/apache/bookkeeper/issues/3792 -- 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] [bookkeeper] hangc0276 opened a new pull request, #3796: BP61: revert BP-59 to release ByteBuf using ReferenceCountUtil.release() instead of ReferenceCountUtil.safeRelease()

2023-02-21 Thread via GitHub
hangc0276 opened a new pull request, #3796: URL: https://github.com/apache/bookkeeper/pull/3796 Fix #3792 -- 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] [bookkeeper] horizonzy commented on a diff in pull request #3794: Pick the higher leak detection level between netty and bookkeeper.

2023-02-21 Thread via GitHub
horizonzy commented on code in PR #3794: URL: https://github.com/apache/bookkeeper/pull/3794#discussion_r1113762084 ## bookkeeper-common-allocator/src/main/java/org/apache/bookkeeper/common/allocator/LeakDetectionPolicy.java: ## @@ -43,5 +43,15 @@ public enum

[GitHub] [bookkeeper] horizonzy commented on a diff in pull request #3794: Pick the higher leak detection level between netty and bookkeeper.

2023-02-21 Thread via GitHub
horizonzy commented on code in PR #3794: URL: https://github.com/apache/bookkeeper/pull/3794#discussion_r1113762084 ## bookkeeper-common-allocator/src/main/java/org/apache/bookkeeper/common/allocator/LeakDetectionPolicy.java: ## @@ -43,5 +43,15 @@ public enum

[GitHub] [bookkeeper] codecov-commenter commented on pull request #3794: Pick the higher leak detection level between netty and bookkeeper.

2023-02-21 Thread via GitHub
codecov-commenter commented on PR #3794: URL: https://github.com/apache/bookkeeper/pull/3794#issuecomment-1439349000 #

[GitHub] [bookkeeper] Shoothzj merged pull request #3795: Upgrade RocksDB version to 7.9.2

2023-02-21 Thread via GitHub
Shoothzj merged PR #3795: URL: https://github.com/apache/bookkeeper/pull/3795 -- 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 (58cdd499c6 -> 80d3aac30f)

2023-02-21 Thread shoothzj
This is an automated email from the ASF dual-hosted git repository. shoothzj pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/bookkeeper.git from 58cdd499c6 Make profile code-coverage and dev-debug tests using `-Dio.netty.leakDetection.level=paranoid`.

[bookkeeper] branch master updated (9ff2954430 -> 58cdd499c6)

2023-02-21 Thread yong
This is an automated email from the ASF dual-hosted git repository. yong pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/bookkeeper.git from 9ff2954430 Add logs for ensemble select failed (#3779) add 58cdd499c6 Make profile code-coverage and dev-debug

[GitHub] [bookkeeper] zymap merged pull request #3785: Make profile `code-coverage` and `dev-debug` tests using `-Dio.netty.leakDetection.level=paranoid`

2023-02-21 Thread via GitHub
zymap merged PR #3785: URL: https://github.com/apache/bookkeeper/pull/3785 -- 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] [bookkeeper] zymap commented on a diff in pull request #3789: Add directIO entry logger settings documents

2023-02-21 Thread via GitHub
zymap commented on code in PR #3789: URL: https://github.com/apache/bookkeeper/pull/3789#discussion_r1113728208 ## site3/website/docs/reference/config.md: ## @@ -156,6 +156,17 @@ The table below lists parameters that you can set to configure bookies. All conf |

[bookkeeper] branch branch-4.15 updated: Add logs for ensemble select failed (#3779)

2023-02-21 Thread yong
This is an automated email from the ASF dual-hosted git repository. yong pushed a commit to branch branch-4.15 in repository https://gitbox.apache.org/repos/asf/bookkeeper.git The following commit(s) were added to refs/heads/branch-4.15 by this push: new 73b0b9fb28 Add logs for ensemble

[bookkeeper] branch master updated: Add logs for ensemble select failed (#3779)

2023-02-21 Thread yong
This is an automated email from the ASF dual-hosted git repository. yong 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 9ff2954430 Add logs for ensemble select

[GitHub] [bookkeeper] zymap merged pull request #3779: Add logs for ensemble select failed

2023-02-21 Thread via GitHub
zymap merged PR #3779: URL: https://github.com/apache/bookkeeper/pull/3779 -- 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] [bookkeeper] merlimat merged pull request #3784: Made PendingAddOp thread safe

2023-02-21 Thread via GitHub
merlimat merged PR #3784: URL: https://github.com/apache/bookkeeper/pull/3784 -- 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: Made PendingAddOp thread safe (#3784)

2023-02-21 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli 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 c295f1880f Made PendingAddOp thread safe

[bookkeeper] branch master updated: change directIO configuration names to avoid confuse (#3791)

2023-02-21 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli 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 5fced0d434 change directIO configuration

[GitHub] [bookkeeper] merlimat merged pull request #3791: Change directIO configuration names to avoid confusing

2023-02-21 Thread via GitHub
merlimat merged PR #3791: URL: https://github.com/apache/bookkeeper/pull/3791 -- 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: Fix CI is not failed when test have errors (#3790)

2023-02-21 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli 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 6b6ea76201 Fix CI is not failed when test

[GitHub] [bookkeeper] merlimat merged pull request #3790: Fix CI is not failed when test have errors

2023-02-21 Thread via GitHub
merlimat merged PR #3790: URL: https://github.com/apache/bookkeeper/pull/3790 -- 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] [bookkeeper] eolivelli commented on a diff in pull request #3794: Pick the higher leak detection level between netty and bookkeeper.

2023-02-21 Thread via GitHub
eolivelli commented on code in PR #3794: URL: https://github.com/apache/bookkeeper/pull/3794#discussion_r1113262055 ## bookkeeper-common-allocator/src/main/java/org/apache/bookkeeper/common/allocator/LeakDetectionPolicy.java: ## @@ -43,5 +43,15 @@ public enum

[GitHub] [bookkeeper] codecov-commenter commented on pull request #3795: Upgrade RocksDB version to 7.9.2

2023-02-21 Thread via GitHub
codecov-commenter commented on PR #3795: URL: https://github.com/apache/bookkeeper/pull/3795#issuecomment-1438658495 #

[GitHub] [bookkeeper] hangc0276 opened a new pull request, #3795: Upgrade RocksDB version to 7.9.2

2023-02-21 Thread via GitHub
hangc0276 opened a new pull request, #3795: URL: https://github.com/apache/bookkeeper/pull/3795 ### Motivation Related to https://github.com/apache/bookkeeper/issues/3734 ### Modification Upgrade RocksDB version to 7.9.2 -- This is an automated message from the Apache Git

[GitHub] [bookkeeper] horizonzy opened a new pull request, #3794: Pick the higher leak detection level between netty and bookkeeper.

2023-02-21 Thread via GitHub
horizonzy opened a new pull request, #3794: URL: https://github.com/apache/bookkeeper/pull/3794 Descriptions of the changes in this PR: ### Motivation Pick the higher leak detection level between netty and bookkeeper. There are detailed information about it:

[GitHub] [bookkeeper] codecov-commenter commented on pull request #3793: Make RrocksDB checksum type configurable

2023-02-21 Thread via GitHub
codecov-commenter commented on PR #3793: URL: https://github.com/apache/bookkeeper/pull/3793#issuecomment-1438146615 #

[GitHub] [bookkeeper] hangc0276 opened a new pull request, #3793: Make RrocksDB checksum type configurable

2023-02-21 Thread via GitHub
hangc0276 opened a new pull request, #3793: URL: https://github.com/apache/bookkeeper/pull/3793 ### Motivation Fix https://github.com/apache/bookkeeper/issues/3734#issuecomment-1407626941 We have two rocksDB tables, one for the ledger index, and another for the entry log location.