[GitHub] sijie commented on issue #1814: Introduce condensed encoding format for availabilityOfEntriesOfLedger

2018-11-16 Thread GitBox
sijie commented on issue #1814: Introduce condensed encoding format for availabilityOfEntriesOfLedger URL: https://github.com/apache/bookkeeper/pull/1814#issuecomment-439586293 run bookkeeper-server bookie tests This is an

[GitHub] nicmichael commented on issue #1808: Allow to configure sticky reads

2018-11-16 Thread GitBox
nicmichael commented on issue #1808: Allow to configure sticky reads URL: https://github.com/apache/bookkeeper/pull/1808#issuecomment-439558592 I actually like @dlg99's suggestion: if each W adjacent entries have the same primary bookie, but the next W entries another primary bookie, we'd

[GitHub] sijie commented on issue #1814: Introduce condensed encoding format for availabilityOfEntriesOfLedger

2018-11-16 Thread GitBox
sijie commented on issue #1814: Introduce condensed encoding format for availabilityOfEntriesOfLedger URL: https://github.com/apache/bookkeeper/pull/1814#issuecomment-439547096 run bookkeeper-server bookie tests This is an

[GitHub] sijie commented on a change in pull request #1812: Allow to configure BK for low latency busy-wait settings

2018-11-16 Thread GitBox
sijie commented on a change in pull request #1812: Allow to configure BK for low latency busy-wait settings URL: https://github.com/apache/bookkeeper/pull/1812#discussion_r234364898 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/BookKeeper.java

[GitHub] sijie commented on a change in pull request #1812: Allow to configure BK for low latency busy-wait settings

2018-11-16 Thread GitBox
sijie commented on a change in pull request #1812: Allow to configure BK for low latency busy-wait settings URL: https://github.com/apache/bookkeeper/pull/1812#discussion_r234364746 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/Journal.java ##

[GitHub] sijie commented on a change in pull request #1812: Allow to configure BK for low latency busy-wait settings

2018-11-16 Thread GitBox
sijie commented on a change in pull request #1812: Allow to configure BK for low latency busy-wait settings URL: https://github.com/apache/bookkeeper/pull/1812#discussion_r234364632 ## File path:

[GitHub] sijie commented on a change in pull request #1812: Allow to configure BK for low latency busy-wait settings

2018-11-16 Thread GitBox
sijie commented on a change in pull request #1812: Allow to configure BK for low latency busy-wait settings URL: https://github.com/apache/bookkeeper/pull/1812#discussion_r234364502 ## File path:

[GitHub] sijie commented on issue #1809: Change LedgerManager to use CompletableFuture

2018-11-16 Thread GitBox
sijie commented on issue #1809: Change LedgerManager to use CompletableFuture URL: https://github.com/apache/bookkeeper/pull/1809#issuecomment-439545892 run bookkeeper-server replication tests This is an automated message

[GitHub] sijie commented on issue #1809: Change LedgerManager to use CompletableFuture

2018-11-16 Thread GitBox
sijie commented on issue #1809: Change LedgerManager to use CompletableFuture URL: https://github.com/apache/bookkeeper/pull/1809#issuecomment-439545823 run bookkeeper replication tests run integration tests This is an

[GitHub] sijie commented on issue #1755: Configure Netty allocator in bookie and client

2018-11-16 Thread GitBox
sijie commented on issue #1755: Configure Netty allocator in bookie and client URL: https://github.com/apache/bookkeeper/pull/1755#issuecomment-439544955 @merlimat can you rebase this pull request to latest master? This is

[GitHub] sijie closed pull request #1807: [bookie] Fix sorted ledger storage rotating entry log files too frequent

2018-11-16 Thread GitBox
sijie closed pull request #1807: [bookie] Fix sorted ledger storage rotating entry log files too frequent URL: https://github.com/apache/bookkeeper/pull/1807 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] sijie commented on issue #1807: [bookie] Fix sorted ledger storage rotating entry log files too frequent

2018-11-16 Thread GitBox
sijie commented on issue #1807: [bookie] Fix sorted ledger storage rotating entry log files too frequent URL: https://github.com/apache/bookkeeper/pull/1807#issuecomment-439544616 IGNORE IT CI This is an automated message

[GitHub] sijie closed pull request #1816: Fix cpu-affinity module build on jdk10+

2018-11-16 Thread GitBox
sijie closed pull request #1816: Fix cpu-affinity module build on jdk10+ URL: https://github.com/apache/bookkeeper/pull/1816 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] sijie commented on issue #1816: Fix cpu-affinity module build on jdk10+

2018-11-16 Thread GitBox
sijie commented on issue #1816: Fix cpu-affinity module build on jdk10+ URL: https://github.com/apache/bookkeeper/pull/1816#issuecomment-439544299 IGNORE IT CI This is an automated message from the Apache Git Service. To

[GitHub] sijie commented on issue #1813: Set default sizes of DbLedgerStorage read and write cache to be propo…

2018-11-16 Thread GitBox
sijie commented on issue #1813: Set default sizes of DbLedgerStorage read and write cache to be propo… URL: https://github.com/apache/bookkeeper/pull/1813#issuecomment-439543810 the reason why it is in blacklist is internal classes are not encouraged to use in general, if there are

[GitHub] eolivelli commented on a change in pull request #1813: Set default sizes of DbLedgerStorage read and write cache to be propo…

2018-11-16 Thread GitBox
eolivelli commented on a change in pull request #1813: Set default sizes of DbLedgerStorage read and write cache to be propo… URL: https://github.com/apache/bookkeeper/pull/1813#discussion_r234360476 ## File path:

[GitHub] merlimat commented on issue #1813: Set default sizes of DbLedgerStorage read and write cache to be propo…

2018-11-16 Thread GitBox
merlimat commented on issue #1813: Set default sizes of DbLedgerStorage read and write cache to be propo… URL: https://github.com/apache/bookkeeper/pull/1813#issuecomment-439541827 > What do you think? Do you know why it was blacklisted? I have no idea why it was blacklisted. I

[GitHub] merlimat commented on a change in pull request #1813: Set default sizes of DbLedgerStorage read and write cache to be propo…

2018-11-16 Thread GitBox
merlimat commented on a change in pull request #1813: Set default sizes of DbLedgerStorage read and write cache to be propo… URL: https://github.com/apache/bookkeeper/pull/1813#discussion_r234294705 ## File path:

[GitHub] eolivelli commented on a change in pull request #1813: Set default sizes of DbLedgerStorage read and write cache to be propo…

2018-11-16 Thread GitBox
eolivelli commented on a change in pull request #1813: Set default sizes of DbLedgerStorage read and write cache to be propo… URL: https://github.com/apache/bookkeeper/pull/1813#discussion_r234292332 ## File path:

[GitHub] merlimat commented on a change in pull request #1813: Set default sizes of DbLedgerStorage read and write cache to be propo…

2018-11-16 Thread GitBox
merlimat commented on a change in pull request #1813: Set default sizes of DbLedgerStorage read and write cache to be propo… URL: https://github.com/apache/bookkeeper/pull/1813#discussion_r234287351 ## File path:

[GitHub] ivankelly commented on a change in pull request #1809: Change LedgerManager to use CompletableFuture

2018-11-16 Thread GitBox
ivankelly commented on a change in pull request #1809: Change LedgerManager to use CompletableFuture URL: https://github.com/apache/bookkeeper/pull/1809#discussion_r234236380 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/BKException.java ##

Jenkins build became unstable: bookkeeper_codecoverage #271

2018-11-16 Thread Apache Jenkins Server
See

Jenkins build is still unstable: bookkeeper_release_branch_48_java8 #93

2018-11-16 Thread Apache Jenkins Server
See

[GitHub] ivankelly commented on a change in pull request #1809: Change LedgerManager to use CompletableFuture

2018-11-16 Thread GitBox
ivankelly commented on a change in pull request #1809: Change LedgerManager to use CompletableFuture URL: https://github.com/apache/bookkeeper/pull/1809#discussion_r234229005 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/BookKeeperAdmin.java

[GitHub] ivankelly commented on a change in pull request #1809: Change LedgerManager to use CompletableFuture

2018-11-16 Thread GitBox
ivankelly commented on a change in pull request #1809: Change LedgerManager to use CompletableFuture URL: https://github.com/apache/bookkeeper/pull/1809#discussion_r234228770 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/BKException.java ##

[GitHub] ivankelly commented on a change in pull request #1809: Change LedgerManager to use CompletableFuture

2018-11-16 Thread GitBox
ivankelly commented on a change in pull request #1809: Change LedgerManager to use CompletableFuture URL: https://github.com/apache/bookkeeper/pull/1809#discussion_r234228416 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieShell.java ##

Jenkins build is still unstable: bookkeeper_release_branch_48_java9 #93

2018-11-16 Thread Apache Jenkins Server
See

[GitHub] ivankelly commented on a change in pull request #1809: Change LedgerManager to use CompletableFuture

2018-11-16 Thread GitBox
ivankelly commented on a change in pull request #1809: Change LedgerManager to use CompletableFuture URL: https://github.com/apache/bookkeeper/pull/1809#discussion_r234209071 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/meta/CleanupLedgerManager.java

Build failed in Jenkins: bookkeeper_release_branch_48_integrationtests #93

2018-11-16 Thread Apache Jenkins Server
See -- [...truncated 1.42 MB...] 2018-11-16\T\13:18:13.926 [INFO] 2018-11-16\T\13:18:13.926 [INFO] --- maven-resources-plugin:2.7:testResources

Build failed in Jenkins: bookkeeper_release_nightly_snapshot #196

2018-11-16 Thread Apache Jenkins Server
See -- Started by timer [EnvInject] - Loading node environment variables. Building remotely on H33 (ubuntu xenial) in workspace

Build failed in Jenkins: bookkeeper_release_branch_47_integrationtests #208

2018-11-16 Thread Apache Jenkins Server
See -- [...truncated 12.22 KB...] message+ 1120 1 1120 Jul 25 0.0 0.0 dbus-daemon --system --fork root 1132 1 1132 Jul 25 0.0 0.0

Build failed in Jenkins: bookkeeper_postcommit_master_python #38

2018-11-16 Thread Apache Jenkins Server
See -- [...truncated 209.05 KB...] jenkins 25409 25407 22909 12:31:09 0.0 0.0 bash .test-infra/scripts/pre-docker-tests.sh jenkins 25412 25409 22909 12:31:09 0.0