[GitHub] reddycharan opened a new pull request #1228: Issue #570: Move logic of unpersisitedbytes to bufferedchannel

reddycharan opened a new pull request #1228: Issue #570: Move logic of unpersisitedbytes to bufferedchannel URL: https://github.com/apache/bookkeeper/pull/1228 Descriptions of the changes in this PR: This is < sub-task2 > of Issue #570.

[GitHub] reddycharan commented on a change in pull request #1225: Issue #570: getting rid of unnecessary synchronization in InterleavedLedgerStorage

reddycharan commented on a change in pull request #1225: Issue #570: getting rid of unnecessary synchronization in InterleavedLedgerStorage URL: https://github.com/apache/bookkeeper/pull/1225#discussion_r172101884 ## File path:

[GitHub] reddycharan commented on a change in pull request #1225: Issue #570: getting rid of unnecessary synchronization in InterleavedLedgerStorage

reddycharan commented on a change in pull request #1225: Issue #570: getting rid of unnecessary synchronization in InterleavedLedgerStorage URL: https://github.com/apache/bookkeeper/pull/1225#discussion_r172101284 ## File path:

[GitHub] sijie closed pull request #1227: Fix AbstractLedgerManagerFactory on handling both shaded class and original class co-exists case

sijie closed pull request #1227: Fix AbstractLedgerManagerFactory on handling both shaded class and original class co-exists case URL: https://github.com/apache/bookkeeper/pull/1227 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] sijie commented on issue #1227: Fix AbstractLedgerManagerFactory on handling both shaded class and original class co-exists case

sijie commented on issue #1227: Fix AbstractLedgerManagerFactory on handling both shaded class and original class co-exists case URL: https://github.com/apache/bookkeeper/pull/1227#issuecomment-370293041 retest this please

[GitHub] jiazhai commented on a change in pull request #1225: Issue #570: getting rid of unnecessary synchronization in InterleavedLedgerStorage

jiazhai commented on a change in pull request #1225: Issue #570: getting rid of unnecessary synchronization in InterleavedLedgerStorage URL: https://github.com/apache/bookkeeper/pull/1225#discussion_r172075243 ## File path:

[GitHub] sijie commented on issue #1227: Fix AbstractLedgerManagerFactory on handling both shaded class and original class co-exists case

sijie commented on issue #1227: Fix AbstractLedgerManagerFactory on handling both shaded class and original class co-exists case URL: https://github.com/apache/bookkeeper/pull/1227#issuecomment-370273602 shading is a bit tricky on testing. so I didn't attempt to add a test case for this.

[GitHub] sijie opened a new pull request #1227: Fix AbstractLedgerManagerFactory on handling both shaded class and original class co-exists case

sijie opened a new pull request #1227: Fix AbstractLedgerManagerFactory on handling both shaded class and original class co-exists case URL: https://github.com/apache/bookkeeper/pull/1227 Descriptions of the changes in this PR: *Problem* #1217 addresses the issue when using

[GitHub] jiazhai closed pull request #1012: [Merge Yahoo repo] CMS-1437: use BookieSocketAddress in DNSToSwitchMapping

jiazhai closed pull request #1012: [Merge Yahoo repo] CMS-1437: use BookieSocketAddress in DNSToSwitchMapping URL: https://github.com/apache/bookkeeper/pull/1012 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] jiazhai commented on issue #1012: [Merge Yahoo repo] CMS-1437: use BookieSocketAddress in DNSToSwitchMapping

jiazhai commented on issue #1012: [Merge Yahoo repo] CMS-1437: use BookieSocketAddress in DNSToSwitchMapping URL: https://github.com/apache/bookkeeper/pull/1012#issuecomment-370273434 will close it. This is an automated

[GitHub] sijie closed pull request #1226: Make table service client includes table service related jars

sijie closed pull request #1226: Make table service client includes table service related jars URL: https://github.com/apache/bookkeeper/pull/1226 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:

[GitHub] sijie opened a new pull request #1226: Make table service client includes table service related jars

sijie opened a new pull request #1226: Make table service client includes table service related jars URL: https://github.com/apache/bookkeeper/pull/1226 Descriptions of the changes in this PR: This is a change for #1205 *Motivation* in order to implify the other

[GitHub] sijie closed pull request #1216: Integrate table service in bookkeeper build

sijie closed pull request #1216: Integrate table service in bookkeeper build URL: https://github.com/apache/bookkeeper/pull/1216 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

Build failed in Jenkins: distributedlog-release-nightly-snapshot #185

See -- [...truncated 298.73 KB...] [INFO] Downloaded: https://repo.maven.apache.org/maven2/io/netty/netty-all/4.1.12.Final/netty-all-4.1.12.Final.pom (25 kB at