Jenkins build is back to stable : distributedlog-nightly-build #487

2017-11-14 Thread Apache Jenkins Server
See

[GitHub] jiazhai opened a new issue #726: Unit Tests failure with BKMetadataVersionException

2017-11-14 Thread GitBox
jiazhai opened a new issue #726: Unit Tests failure with BKMetadataVersionException URL: https://github.com/apache/bookkeeper/issues/726 **BUG REPORT** 1. Please describe the issue you observed: - What did you do? Run mvn test - What did you expect to see? Expect all

[GitHub] sijie closed issue #613: Bookie: LRU FileInfo Eviction Policy

2017-11-14 Thread GitBox
sijie closed issue #613: Bookie: LRU FileInfo Eviction Policy URL: https://github.com/apache/bookkeeper/issues/613 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[GitHub] sijie commented on issue #613: Bookie: LRU FileInfo Eviction Policy

2017-11-14 Thread GitBox
sijie commented on issue #613: Bookie: LRU FileInfo Eviction Policy URL: https://github.com/apache/bookkeeper/issues/613#issuecomment-344420235 This issue is already addressed by #513 This is an automated message from the

[GitHub] sijie commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie

2017-11-14 Thread GitBox
sijie commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie URL: https://github.com/apache/bookkeeper/pull/712#discussion_r150927573 ## File path:

[GitHub] sijie commented on issue #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie

2017-11-14 Thread GitBox
sijie commented on issue #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie URL: https://github.com/apache/bookkeeper/pull/712#issuecomment-344357571 @ivankelly I checked your changes. Frankly speaking, I don't see any real difference comparing to

[GitHub] ivankelly commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie

2017-11-14 Thread GitBox
ivankelly commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie URL: https://github.com/apache/bookkeeper/pull/712#discussion_r150924748 ## File path:

[GitHub] ivankelly commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie

2017-11-14 Thread GitBox
ivankelly commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie URL: https://github.com/apache/bookkeeper/pull/712#discussion_r150923743 ## File path:

[GitHub] sijie commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie

2017-11-14 Thread GitBox
sijie commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie URL: https://github.com/apache/bookkeeper/pull/712#discussion_r150916980 ## File path:

[GitHub] ivankelly commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie

2017-11-14 Thread GitBox
ivankelly commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie URL: https://github.com/apache/bookkeeper/pull/712#discussion_r150808708 ## File path:

[GitHub] ivankelly commented on issue #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie

2017-11-14 Thread GitBox
ivankelly commented on issue #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie URL: https://github.com/apache/bookkeeper/pull/712#issuecomment-344231541 I created a pull request into your branch. https://github.com/sijie/bookkeeper/pull/1

[GitHub] sijie commented on issue #553: Documentation for new API

2017-11-14 Thread GitBox
sijie commented on issue #553: Documentation for new API URL: https://github.com/apache/bookkeeper/issues/553#issuecomment-344217221 @eolivelli we can only remove old API after we deprecate the old api. This is an automated

[GitHub] sijie commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie

2017-11-14 Thread GitBox
sijie commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie URL: https://github.com/apache/bookkeeper/pull/712#discussion_r150789467 ## File path:

[GitHub] sijie commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie

2017-11-14 Thread GitBox
sijie commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie URL: https://github.com/apache/bookkeeper/pull/712#discussion_r150787234 ## File path:

[GitHub] sijie commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie

2017-11-14 Thread GitBox
sijie commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie URL: https://github.com/apache/bookkeeper/pull/712#discussion_r150785358 ## File path:

[GitHub] ivankelly commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie

2017-11-14 Thread GitBox
ivankelly commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie URL: https://github.com/apache/bookkeeper/pull/712#discussion_r150784588 ## File path:

[GitHub] ivankelly commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie

2017-11-14 Thread GitBox
ivankelly commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie URL: https://github.com/apache/bookkeeper/pull/712#discussion_r150781772 ## File path:

[GitHub] sijie commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie

2017-11-14 Thread GitBox
sijie commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie URL: https://github.com/apache/bookkeeper/pull/712#discussion_r150778486 ## File path:

[GitHub] jiazhai closed pull request #725: Issue 724: fix wrong link for slack signup

2017-11-14 Thread GitBox
jiazhai closed pull request #725: Issue 724: fix wrong link for slack signup URL: https://github.com/apache/bookkeeper/pull/725 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] ivankelly commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie

2017-11-14 Thread GitBox
ivankelly commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie URL: https://github.com/apache/bookkeeper/pull/712#discussion_r150778031 ## File path:

[GitHub] jiazhai closed issue #724: Slack signup link is wrong

2017-11-14 Thread GitBox
jiazhai closed issue #724: Slack signup link is wrong URL: https://github.com/apache/bookkeeper/issues/724 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the

[GitHub] eolivelli commented on issue #553: Documentation for new API

2017-11-14 Thread GitBox
eolivelli commented on issue #553: Documentation for new API URL: https://github.com/apache/bookkeeper/issues/553#issuecomment-344195264 great, I will add some paragraphs, as the new API is still "experimental" is it better to leave the old API on foreground, OK ?

[GitHub] ivankelly commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie

2017-11-14 Thread GitBox
ivankelly commented on a change in pull request #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie URL: https://github.com/apache/bookkeeper/pull/712#discussion_r150773274 ## File path:

[GitHub] jiazhai commented on issue #724: Slack signup link is wrong

2017-11-14 Thread GitBox
jiazhai commented on issue #724: Slack signup link is wrong URL: https://github.com/apache/bookkeeper/issues/724#issuecomment-344191812 Thanks @sijie for reporting this. PR #725 opened for it. This is an automated message

[GitHub] eolivelli commented on issue #553: Documentation for new API

2017-11-14 Thread GitBox
eolivelli commented on issue #553: Documentation for new API URL: https://github.com/apache/bookkeeper/issues/553#issuecomment-344191519 I will do my best. Can you give some hint? I think that a bunch of examples and a link to javadocs (how can I do this?) will be enough

[GitHub] sijie opened a new issue #724: Slack signup link is wrong

2017-11-14 Thread GitBox
sijie opened a new issue #724: Slack signup link is wrong URL: https://github.com/apache/bookkeeper/issues/724 **BUG REPORT** 1. Please describe the issue you observed: - What did you do? Open https://bookkeeper.apache.org/community/slack/ and click the

[GitHub] sijie opened a new issue #723: Storing ledger metadata in binary format

2017-11-14 Thread GitBox
sijie opened a new issue #723: Storing ledger metadata in binary format URL: https://github.com/apache/bookkeeper/issues/723 **FEATURE REQUEST** 1. Please describe the feature you are requesting. Currently we stores ledger metadata in TextFormat protobuf not in binary

Jenkins build is still unstable: bookkeeper-release-nightly-snapshot #102

2017-11-14 Thread Apache Jenkins Server
See