Re: BP-11: New BookKeeper Website

2017-07-28 Thread Jia Zhai
It is a great work, Agree that we merge this first, since it won't affect old site. Regarding the small suggestion, same feeling of these: 1, on Mobile device(or narrow your web browser on PC), the tab on the top right conner could work. 2, the first page may need contains more information.

Build failed in Jenkins: bookkeeper-master #1831

2017-07-28 Thread Apache Jenkins Server
See Changes: [jiazhai] ISSUE #308: LocalBookKeeper.java rethrow Exception in case of failed [jiazhai] ISSUE #306: Update SCM info for using mvn:release [jiazhai] ISSUE #296: Bookie supports ephemeral port

[jira] [Commented] (BOOKKEEPER-1104) BookieInitializationTest.testWithDiskFullAndAbilityToCreateNewIndexFile testcase is unreliable

2017-07-28 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1104?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16104994#comment-16104994 ] Hudson commented on BOOKKEEPER-1104: FAILURE: Integrated in Jenkins build bookkeeper-master #1831

[GitHub] eolivelli opened a new pull request #317: Issue-316 BookKeeperTest#testReadEntryReleaseByteBufs is failing

2017-07-28 Thread git
eolivelli opened a new pull request #317: Issue-316 BookKeeperTest#testReadEntryReleaseByteBufs is failing URL: https://github.com/apache/bookkeeper/pull/317 Change the assertion in BookKeeperTest#testReadEntryReleaseByteBufs: expect a NullPointerException instead of a

New used classes in 4.5 are deprecated in Java9

2017-07-28 Thread Enrico Olivelli
Hi all, I'm back to tring to build and run BookKeeper on java9 recently we added extensive usage of java.uti.Observable and Observer. This classes have been deprecated in java9 and the build fails, because we forbid the usage of deprecated APIs. These classes are not used on client API but only

[GitHub] eolivelli commented on issue #317: Issue-316 BookKeeperTest#testReadEntryReleaseByteBufs is failing

2017-07-28 Thread git
eolivelli commented on issue #317: Issue-316 BookKeeperTest#testReadEntryReleaseByteBufs is failing URL: https://github.com/apache/bookkeeper/pull/317#issuecomment-318665060 There is one problem, I have chagne the assertion and if I run the single test it works as expected, but if I run

Re: [VOTE] Slack Channel for BookKeeper

2017-07-28 Thread Venkateswara Rao Jujjuri
+1 Just curious on why decided to transfer rather than create a new one. Fine either way. On Fri, Jul 28, 2017 at 12:22 AM, Francesco Caliumi - Diennea < francesco.cali...@diennea.com> wrote: > +1 non binding > > On Thu, 2017-07-27 at 21:45 +0200, Ivan Kelly wrote: > > +1 > > On Thu, Jul 27,

[GitHub] zhaijack opened a new pull request #315: Issue 314: Enable checkstyle on bookkeeper-server

2017-07-28 Thread git
zhaijack opened a new pull request #315: Issue 314: Enable checkstyle on bookkeeper-server URL: https://github.com/apache/bookkeeper/pull/315 Descriptions of the changes in this PR: - enable checkstyle plugin in bookkeeper-server with a suppression configuration file - enable

Re: BookKeeper 4.5.0 performance regression ?

2017-07-28 Thread Enrico Olivelli
Another piece of this story I have mocked a dummy LedgerStorage (like DummyLedgerStorage in BK tests) which does nothing (no real disk storage, non memory, no index), no checkpoint The problems happens anyway, after a bunch opf seconds (20-30s) the system slows down So the storage is not the

[GitHub] eolivelli commented on issue #313: Issue 300: EnsemblePlacementPolicy must not use com.google.common.base.Optional but java.util.Optional

2017-07-28 Thread git
eolivelli commented on issue #313: Issue 300: EnsemblePlacementPolicy must not use com.google.common.base.Optional but java.util.Optional URL: https://github.com/apache/bookkeeper/pull/313#issuecomment-318600647 @zhaijack I have already sent a PR #302 I am sorry for this duplicate

[GitHub] eolivelli commented on issue #313: Issue 300: EnsemblePlacementPolicy must not use com.google.common.base.Optional but java.util.Optional

2017-07-28 Thread git
eolivelli commented on issue #313: Issue 300: EnsemblePlacementPolicy must not use com.google.common.base.Optional but java.util.Optional URL: https://github.com/apache/bookkeeper/pull/313#issuecomment-318600828 we can go with your patch if you prefer @merlimat already approved my pr and

[GitHub] zhaijack commented on issue #313: Issue 300: EnsemblePlacementPolicy must not use com.google.common.base.Optional but java.util.Optional

2017-07-28 Thread git
zhaijack commented on issue #313: Issue 300: EnsemblePlacementPolicy must not use com.google.common.base.Optional but java.util.Optional URL: https://github.com/apache/bookkeeper/pull/313#issuecomment-318605089 Oh, Sorry for it. my bad. I will close mine.

[GitHub] zhaijack closed pull request #313: Issue 300: EnsemblePlacementPolicy must not use com.google.common.base.Optional but java.util.Optional

2017-07-28 Thread git
zhaijack closed pull request #313: Issue 300: EnsemblePlacementPolicy must not use com.google.common.base.Optional but java.util.Optional URL: https://github.com/apache/bookkeeper/pull/313 This is an automated message from

[GitHub] jiazhai closed issue #300: EnsemblePlacementPolicy must not use com.google.common.base.Optional but java.util.Optional

2017-07-28 Thread git
jiazhai closed issue #300: EnsemblePlacementPolicy must not use com.google.common.base.Optional but java.util.Optional URL: https://github.com/apache/bookkeeper/issues/300 This is an automated message from the Apache Git

[GitHub] jiazhai closed pull request #302: Issue-300 EnsemblePlacementPolicy must not use com.google.common.base.Optional but java.util.Optional

2017-07-28 Thread git
jiazhai closed pull request #302: Issue-300 EnsemblePlacementPolicy must not use com.google.common.base.Optional but java.util.Optional URL: https://github.com/apache/bookkeeper/pull/302 This is an automated message from

[GitHub] sijie opened a new issue #319: Release 4.5.0

2017-07-28 Thread git
sijie opened a new issue #319: Release 4.5.0 URL: https://github.com/apache/bookkeeper/issues/319 Tracking the checklist for completing release 4.5.0 - [] Release Notes (feature highlights) - [] Configuration, Documentation for new features - [] Upgrade story - []

[GitHub] sijie commented on issue #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-28 Thread git
sijie commented on issue #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#issuecomment-318740951 @eolivelli : @yzang already addressed your comments, can you review it again? @dcastor : can you review @yzang 's

[GitHub] sijie closed issue #294: BP-11: New BooKeeper Website

2017-07-28 Thread git
sijie closed issue #294: BP-11: New BooKeeper Website URL: https://github.com/apache/bookkeeper/issues/294 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use

[GitHub] sijie commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image

2017-07-28 Thread git
sijie commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#issuecomment-318753769 @caiok @jiazhai as we discussed in the community meeting, do you guys want to work together to move this forward?

[jira] [Resolved] (BOOKKEEPER-1103) LedgerMetadataCreateTest bug in ledger id generation causes intermittent hang

2017-07-28 Thread Sijie Guo (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1103?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo resolved BOOKKEEPER-1103. --- Resolution: Fixed Fix Version/s: 4.5.0 Issue resolved by merging pull request 303

[GitHub] sijie commented on issue #80: BOOKKEEPER-816: use native fallocate to improve journal allocation

2017-07-28 Thread git
sijie commented on issue #80: BOOKKEEPER-816: use native fallocate to improve journal allocation URL: https://github.com/apache/bookkeeper/pull/80#issuecomment-318741311 defer this to 4.6.0, because this might potentially make release a bit complicated. it is non 4.5 blocker, we can defer

Release Notes 4.5.0

2017-07-28 Thread Sijie Guo
Hi all, 4.5.0 is coming. Generating the 4.5.0 release notes is a bit tricky for a few reasons. - The 4.5.0 release has a lot of big features addressed. We need to highlight them and make sure we don't miss any big features. - We are currently managing issues on both Github and JIRA.

[GitHub] sijie opened a new issue #318: Documentation Improvements

2017-07-28 Thread git
sijie opened a new issue #318: Documentation Improvements URL: https://github.com/apache/bookkeeper/issues/318 Created a master issue for tracking any questions, improvements regarding the new website/documentation. This is

[GitHub] sijie closed pull request #303: BOOKKEEPER-1103: Fix LedgerMetadataCreateTest random id loop

2017-07-28 Thread git
sijie closed pull request #303: BOOKKEEPER-1103: Fix LedgerMetadataCreateTest random id loop URL: https://github.com/apache/bookkeeper/pull/303 This is an automated message from the Apache Git Service. To respond to the

[GitHub] sijie closed pull request #311: BOOKKEEPER-935: Javadoc improvements and publish sources and javadocs to Maven Central

2017-07-28 Thread git
sijie closed pull request #311: BOOKKEEPER-935: Javadoc improvements and publish sources and javadocs to Maven Central URL: https://github.com/apache/bookkeeper/pull/311 This is an automated message from the Apache Git

[jira] [Resolved] (BOOKKEEPER-935) Publish sources and javadocs to Maven Central

2017-07-28 Thread Sijie Guo (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-935?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo resolved BOOKKEEPER-935. -- Resolution: Fixed Issue resolved by merging pull request 311

Re: New used classes in 4.5 are deprecated in Java9

2017-07-28 Thread Sijie Guo
On Jul 28, 2017 7:34 AM, "Enrico Olivelli" wrote: Hi all, I'm back to tring to build and run BookKeeper on java9 recently we added extensive usage of java.uti.Observable and Observer. This classes have been deprecated in java9 and the build fails, because we forbid the

Re: [VOTE] Slack Channel for BookKeeper

2017-07-28 Thread Sijie Guo
On Fri, Jul 28, 2017 at 8:21 AM, Venkateswara Rao Jujjuri wrote: > +1 > Just curious on why decided to transfer rather than create a new one. > Fine either way. > This came out when we transfers the DL project to BK PMC. The slack team has to be transferred to BK PMC anyway.

[GitHub] kishorekasi commented on issue #183: BOOKKEEPER-588 SSL Support for Bookkeeper

2017-07-28 Thread git
kishorekasi commented on issue #183: BOOKKEEPER-588 SSL Support for Bookkeeper URL: https://github.com/apache/bookkeeper/pull/183#issuecomment-318781532 @merlimat I can change SSL to TLS for class names. But rename variables to tls may not be appropriate as we will still use JDK and Netty

[GitHub] sijie commented on a change in pull request #322: Issue 320: add scripts for publishing new website to staging and apache

2017-07-28 Thread git
sijie commented on a change in pull request #322: Issue 320: add scripts for publishing new website to staging and apache URL: https://github.com/apache/bookkeeper/pull/322#discussion_r130208977 ## File path: site/scripts/publish-website.sh ## @@ -0,0 +1,48 @@

[GitHub] sijie opened a new issue #320: Add script for publishing website to staging and apache

2017-07-28 Thread git
sijie opened a new issue #320: Add script for publishing website to staging and apache URL: https://github.com/apache/bookkeeper/issues/320 Provide a script for publishing website to staging and apache This is an automated

[GitHub] sijie opened a new issue #321: Update "who are we?"

2017-07-28 Thread git
sijie opened a new issue #321: Update "who are we?" URL: https://github.com/apache/bookkeeper/issues/321 http://bookkeeper.apache.org/test/content/project/who/ is a bit out of date, we should update it to reflect current list of pmc members, committers.

[GitHub] lucperkins commented on a change in pull request #322: Issue 320: add scripts for publishing new website to staging and apache

2017-07-28 Thread git
lucperkins commented on a change in pull request #322: Issue 320: add scripts for publishing new website to staging and apache URL: https://github.com/apache/bookkeeper/pull/322#discussion_r130208848 ## File path: site/scripts/publish-website.sh ## @@ -0,0 +1,48 @@

[GitHub] sijie opened a new pull request #322: Issue 320: add scripts for publishing new website to staging and apache

2017-07-28 Thread git
sijie opened a new pull request #322: Issue 320: add scripts for publishing new website to staging and apache URL: https://github.com/apache/bookkeeper/pull/322 Descriptions of the changes in this PR: - script for staging changes to a staging-site repo - steps for how to staging

[GitHub] sijie commented on a change in pull request #322: Issue 320: add scripts for publishing new website to staging and apache

2017-07-28 Thread git
sijie commented on a change in pull request #322: Issue 320: add scripts for publishing new website to staging and apache URL: https://github.com/apache/bookkeeper/pull/322#discussion_r130209287 ## File path: site/scripts/publish-website.sh ## @@ -0,0 +1,48 @@

[GitHub] zhaijack commented on issue #315: Issue 314: Enable checkstyle on bookkeeper-server

2017-07-28 Thread git
zhaijack commented on issue #315: Issue 314: Enable checkstyle on bookkeeper-server URL: https://github.com/apache/bookkeeper/pull/315#issuecomment-318805713 doing it now This is an automated message from the Apache Git

[GitHub] zhaijack commented on issue #315: Issue 314: Enable checkstyle on bookkeeper-server

2017-07-28 Thread git
zhaijack commented on issue #315: Issue 314: Enable checkstyle on bookkeeper-server URL: https://github.com/apache/bookkeeper/pull/315#issuecomment-318805713 Merged with latest code This is an automated message from the

[GitHub] sijie closed issue #320: Add script for publishing website to staging and apache

2017-07-28 Thread git
sijie closed issue #320: Add script for publishing website to staging and apache URL: https://github.com/apache/bookkeeper/issues/320 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] sijie closed pull request #322: Issue 320: add scripts for publishing new website to staging and apache

2017-07-28 Thread git
sijie closed pull request #322: Issue 320: add scripts for publishing new website to staging and apache URL: https://github.com/apache/bookkeeper/pull/322 This is an automated message from the Apache Git Service. To

[GitHub] sijie opened a new issue #324: add redirections for documentation from old releases.

2017-07-28 Thread git
sijie opened a new issue #324: add redirections for documentation from old releases. URL: https://github.com/apache/bookkeeper/issues/324 The new beta website only contains the documentation for latest release. We should have links to point to the documentation for old releases.

Re: BP-11: New BookKeeper Website

2017-07-28 Thread Sijie Guo
Hi all, Thanks Luc for his hard working on the website, and thanks Jia and Matteo for reviewing the beta website. I pushed the changes and made it live under /test/content : http://bookkeeper.apache.org/test/content/ Here is the instructions on how to make changes to website, run locally,

[GitHub] sijie opened a new issue #323: ci job to automatically publish website to apache once the pull request is merged

2017-07-28 Thread git
sijie opened a new issue #323: ci job to automatically publish website to apache once the pull request is merged URL: https://github.com/apache/bookkeeper/issues/323 Pushing website to live should be automated. A CI job can be triggered to do this once the source file change is done.

[GitHub] sijie commented on issue #318: Documentation Improvements

2017-07-28 Thread git
sijie commented on issue #318: Documentation Improvements URL: https://github.com/apache/bookkeeper/issues/318#issuecomment-318793599 /cc @lucperkins for awareness This is an automated message from the Apache Git Service. To

[GitHub] sijie opened a new issue #325: [website] how to manage documentation between releases

2017-07-28 Thread git
sijie opened a new issue #325: [website] how to manage documentation between releases URL: https://github.com/apache/bookkeeper/issues/325 Currently the new website documentation points to latest master. we are lack of the ability for versioning the documentation. this should be also part

[GitHub] jiazhai commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image

2017-07-28 Thread git
jiazhai commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#issuecomment-318795947 @sijie @caiok How about merge this first, and open another issue to improve it following the comments above?

[GitHub] dcastor commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-28 Thread git
dcastor commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r130211263 ## File path: bookkeeper-http/http-server/src/main/java/org/apache/bookkeeper/http/HttpServer.java