Jenkins build is unstable: bookkeeper-master #1841

See

Re: bookkeeper website from cms to git

Il ven 4 ago 2017, 07:56 Sijie Guo ha scritto: > I've done copying the CMS content to Git under `/archives`. That means the > old website will be placed in /archives. > > Here is the result of http://bookkeeper.apache.org/test/content/archives/ > > - '/archives' are pointing to the old CMS conten

[GitHub] sijie opened a new pull request #384: Issue 383: mvn release:branch fails

sijie opened a new pull request #384: Issue 383: mvn release:branch fails URL: https://github.com/apache/bookkeeper/pull/384 Descriptions of the changes in this PR: - change `compilerArguments` to `compilerArgs` - use `compilerArg` for configuring the arguments for compilation. -

[GitHub] sijie opened a new issue #383: mvn release:branch fails

sijie opened a new issue #383: mvn release:branch fails URL: https://github.com/apache/bookkeeper/issues/383 **BUG REPORT** 1. Please describe the issue you observed: - What did you do? @jvrao and me were working together to cut a release branch. - What did you ex

Re: bookkeeper website from cms to git

I've done copying the CMS content to Git under `/archives`. That means the old website will be placed in /archives. Here is the result of http://bookkeeper.apache.org/test/content/archives/ - '/archives' are pointing to the old CMS content - the links in `releases` menus are pointing to '/archive

[GitHub] sijie closed pull request #382: Issue 381: update old releases url to achives

sijie closed pull request #382: Issue 381: update old releases url to achives URL: https://github.com/apache/bookkeeper/pull/382 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] sijie closed issue #381: [website] update old releases url to achives

sijie closed issue #381: [website] update old releases url to achives URL: https://github.com/apache/bookkeeper/issues/381 This is an automated message from the Apache Git Service. To respond to the message, please log on Gi

[GitHub] sijie commented on issue #375: Issue-262 Useless option explicitLacInterval is set to 1 in default bk_server.conf

sijie commented on issue #375: Issue-262 Useless option explicitLacInterval is set to 1 in default bk_server.conf URL: https://github.com/apache/bookkeeper/pull/375#issuecomment-320159177 yes we need to remove it from yaml file if it exists there. -

[GitHub] sijie commented on a change in pull request #382: Issue 381: update old releases url to achives

sijie commented on a change in pull request #382: Issue 381: update old releases url to achives URL: https://github.com/apache/bookkeeper/pull/382#discussion_r131310317 ## File path: site/_data/cli/shell.yaml ## @@ -119,13 +119,12 @@ commands: description: End Positio

[GitHub] jiazhai closed issue #376: Skip getSysFileDescriptor if JournalRemovePagesFromCache=false

jiazhai closed issue #376: Skip getSysFileDescriptor if JournalRemovePagesFromCache=false URL: https://github.com/apache/bookkeeper/issues/376 This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] jiazhai closed pull request #380: Issue-376 Skip getSysFileDescriptor if JournalRemovePagesFromCache=false

jiazhai closed pull request #380: Issue-376 Skip getSysFileDescriptor if JournalRemovePagesFromCache=false URL: https://github.com/apache/bookkeeper/pull/380 This is an automated message from the Apache Git Service. To resp

[GitHub] jiazhai closed issue #269: Upgrade findbugs to 3.0.4

jiazhai closed issue #269: Upgrade findbugs to 3.0.4 URL: https://github.com/apache/bookkeeper/issues/269 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the

[GitHub] jiazhai closed pull request #379: Issue-269 upgrade findbugs to 3.0.4

jiazhai closed pull request #379: Issue-269 upgrade findbugs to 3.0.4 URL: https://github.com/apache/bookkeeper/pull/379 This is an automated message from the Apache Git Service. To respond to the message, please log on GitH

[GitHub] merlimat commented on a change in pull request #382: Issue 381: update old releases url to achives

merlimat commented on a change in pull request #382: Issue 381: update old releases url to achives URL: https://github.com/apache/bookkeeper/pull/382#discussion_r131307861 ## File path: site/_data/cli/shell.yaml ## @@ -119,13 +119,12 @@ commands: description: End Posi

[GitHub] sijie commented on issue #382: Issue 381: update old releases url to achives

sijie commented on issue #382: Issue 381: update old releases url to achives URL: https://github.com/apache/bookkeeper/pull/382#issuecomment-320148906 /cc @lucperkins for reviews This is an automated message from the Apache G

[GitHub] sijie opened a new pull request #382: Issue 381: update old releases url to achives

sijie opened a new pull request #382: Issue 381: update old releases url to achives URL: https://github.com/apache/bookkeeper/pull/382 Descriptions of the changes in this PR: - the cms content has been copied from cms to asf-site branch under /content/archives. it is available under

[GitHub] sijie opened a new issue #381: [website] update old releases url to achives

sijie opened a new issue #381: [website] update old releases url to achives URL: https://github.com/apache/bookkeeper/issues/381 *Problem* the old releases were redirected back to bookkeeper.apache.org/docs/rx.y.z. No it is time to cut over bookkeeper website from CMS to Git.

bookkeeper website from cms to git

Hi all, I had a discussion with INFRA about how to cut over the website from CMS to git. The easier solution they suggested: - copy the CMS content to Git. - cut over the DNS from CMS to Git. - deprecate CMS I will try copy the content to make it live under /test/content. At the same time, I'd l

[GitHub] zhaijack commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions

zhaijack commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions URL: https://github.com/apache/bookkeeper/pull/342#discussion_r131291099 ## File path: docker/scripts/entrypoint.sh ## @@ -0,0 +1,72 @@ +#!/bin/bash

[GitHub] merlimat commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions

merlimat commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions URL: https://github.com/apache/bookkeeper/pull/342#discussion_r131289718 ## File path: docker/scripts/entrypoint.sh ## @@ -0,0 +1,72 @@ +#!/bin/bash

[GitHub] zhaijack commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions

zhaijack commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions URL: https://github.com/apache/bookkeeper/pull/342#discussion_r131288788 ## File path: docker/scripts/entrypoint.sh ## @@ -0,0 +1,72 @@ +#!/bin/bash

[GitHub] zhaijack commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions

zhaijack commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions URL: https://github.com/apache/bookkeeper/pull/342#discussion_r131287631 ## File path: docker/LICENSE ## @@ -0,0 +1,201 @@ +

[GitHub] zhaijack commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions

zhaijack commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions URL: https://github.com/apache/bookkeeper/pull/342#discussion_r131287387 ## File path: docker/README.md ## @@ -0,0 +1,174 @@ + +# What is Apache Boo

[GitHub] zhaijack commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions

zhaijack commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions URL: https://github.com/apache/bookkeeper/pull/342#discussion_r131287358 ## File path: docker/README.md ## @@ -0,0 +1,174 @@ + +# What is Apache Boo

[GitHub] merlimat commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions

merlimat commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions URL: https://github.com/apache/bookkeeper/pull/342#discussion_r131268944 ## File path: docker/README.md ## @@ -0,0 +1,174 @@ + +# What is Apache Boo

[GitHub] merlimat commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions

merlimat commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions URL: https://github.com/apache/bookkeeper/pull/342#discussion_r131269945 ## File path: docker/LICENSE ## @@ -0,0 +1,201 @@ +

[GitHub] merlimat commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions

merlimat commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions URL: https://github.com/apache/bookkeeper/pull/342#discussion_r131269443 ## File path: docker/scripts/entrypoint.sh ## @@ -0,0 +1,72 @@ +#!/bin/bash

[GitHub] merlimat commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions

merlimat commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions URL: https://github.com/apache/bookkeeper/pull/342#discussion_r131270711 ## File path: docker/scripts/entrypoint.sh ## @@ -0,0 +1,72 @@ +#!/bin/bash

[GitHub] merlimat commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions

merlimat commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions URL: https://github.com/apache/bookkeeper/pull/342#discussion_r131269694 ## File path: docker/scripts/entrypoint.sh ## @@ -0,0 +1,72 @@ +#!/bin/bash

[GitHub] merlimat commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions

merlimat commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions URL: https://github.com/apache/bookkeeper/pull/342#discussion_r131270226 ## File path: docker/README.md ## @@ -0,0 +1,174 @@ + +# What is Apache Boo

[GitHub] merlimat closed pull request #373: Issue #372: Allow to configure advertised address in bookies

merlimat closed pull request #373: Issue #372: Allow to configure advertised address in bookies URL: https://github.com/apache/bookkeeper/pull/373 This is an automated message from the Apache Git Service. To respond to the

[GitHub] merlimat closed issue #372: Allow to configure a specific address to advertise the bookie

merlimat closed issue #372: Allow to configure a specific address to advertise the bookie URL: https://github.com/apache/bookkeeper/issues/372 This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] merlimat closed issue #377: Exception in double registering of Prometheus metric collectors

merlimat closed issue #377: Exception in double registering of Prometheus metric collectors URL: https://github.com/apache/bookkeeper/issues/377 This is an automated message from the Apache Git Service. To respond to the me

[GitHub] merlimat closed pull request #378: Issue #377: Make Prometheus stats logger registration idempotent

merlimat closed pull request #378: Issue #377: Make Prometheus stats logger registration idempotent URL: https://github.com/apache/bookkeeper/pull/378 This is an automated message from the Apache Git Service. To respond to

[GitHub] merlimat commented on issue #378: Issue #377: Make Prometheus stats logger registration idempotent

merlimat commented on issue #378: Issue #377: Make Prometheus stats logger registration idempotent URL: https://github.com/apache/bookkeeper/pull/378#issuecomment-320087028 > It is a pity that we need reflection for such tasks. Maybe in future releases of PT client lib the problem will be

[GitHub] eolivelli commented on issue #376: Skip getSysFileDescriptor if JournalRemovePagesFromCache=false

eolivelli commented on issue #376: Skip getSysFileDescriptor if JournalRemovePagesFromCache=false URL: https://github.com/apache/bookkeeper/issues/376#issuecomment-320083348 @sijie I know we are in the code freeze phase, but this change is minimal, can we evaluate to add it into 4.5 ? --

[GitHub] merlimat commented on a change in pull request #373: Issue #372: Allow to configure advertised address in bookies

merlimat commented on a change in pull request #373: Issue #372: Allow to configure advertised address in bookies URL: https://github.com/apache/bookkeeper/pull/373#discussion_r131253373 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/Bookie.java ###

[GitHub] eolivelli opened a new pull request #380: Issue-376 Skip getSysFileDescriptor if JournalRemovePagesFromCache=false

eolivelli opened a new pull request #380: Issue-376 Skip getSysFileDescriptor if JournalRemovePagesFromCache=false URL: https://github.com/apache/bookkeeper/pull/380 Skip illegal reflective calls if not needed, this way the Bookie will run on Java9 cleanly We need to access internal

[GitHub] eolivelli opened a new pull request #379: Issue-269 upgrade findbugs to 3.0.4

eolivelli opened a new pull request #379: Issue-269 upgrade findbugs to 3.0.4 URL: https://github.com/apache/bookkeeper/pull/379 Just update findbugs plugin to latest version. No new warnings to address This is an automated m

[GitHub] merlimat opened a new pull request #378: Issue #377: Make Prometheus stats logger registration idempotent

merlimat opened a new pull request #378: Issue #377: Make Prometheus stats logger registration idempotent URL: https://github.com/apache/bookkeeper/pull/378 Make sure the metric are only registered once on Prometheus client lib. Discusses in #377. --

[GitHub] merlimat opened a new issue #377: Exception in double registering of Prometheus metric collectors

merlimat opened a new issue #377: Exception in double registering of Prometheus metric collectors URL: https://github.com/apache/bookkeeper/issues/377 When the `PrometheusMetricsProvider` is enabled in the bookie, there can be cases in which the metrics are registered multiple times, leadi

[GitHub] eolivelli opened a new issue #376: Skip getSysFileDescriptor if JournalRemovePagesFromCache=false

eolivelli opened a new issue #376: Skip getSysFileDescriptor if JournalRemovePagesFromCache=false URL: https://github.com/apache/bookkeeper/issues/376 This is the most critical show-stopper for running cleanly the Bookie on java9. Waiting for the real solution of the problem, that needs

[GitHub] eolivelli opened a new pull request #375: Issue-262 Useless option explicitLacInterval is set to 1 in default bk_server.conf

eolivelli opened a new pull request #375: Issue-262 Useless option explicitLacInterval is set to 1 in default bk_server.conf URL: https://github.com/apache/bookkeeper/pull/375 Remove an useless line from default bk_server.conf file -

[GitHub] eolivelli commented on issue #375: Issue-262 Useless option explicitLacInterval is set to 1 in default bk_server.conf

eolivelli commented on issue #375: Issue-262 Useless option explicitLacInterval is set to 1 in default bk_server.conf URL: https://github.com/apache/bookkeeper/pull/375#issuecomment-320077603 this is for issue #262 This is

[GitHub] eolivelli commented on issue #373: Issue #372: Allow to configure advertised address in bookies

eolivelli commented on issue #373: Issue #372: Allow to configure advertised address in bookies URL: https://github.com/apache/bookkeeper/pull/373#issuecomment-320074294 Yep @sijie it is what I meant, that for me is ok to have it in 4.5. We are on same page --

[GitHub] sijie commented on issue #373: Issue #372: Allow to configure advertised address in bookies

sijie commented on issue #373: Issue #372: Allow to configure advertised address in bookies URL: https://github.com/apache/bookkeeper/pull/373#issuecomment-320072796 @eolivelli let's squeeze this in 4.5.0. I have same problem when I test 4.5 on google cloud. -

[GitHub] sijie commented on issue #374: Generate the config documentation from bk_server.conf

sijie commented on issue #374: Generate the config documentation from bk_server.conf URL: https://github.com/apache/bookkeeper/issues/374#issuecomment-320072472 /cc @lucperkins This is an automated message from the Apache G

[GitHub] sijie opened a new issue #374: Generate the config documentation from bk_server.conf

sijie opened a new issue #374: Generate the config documentation from bk_server.conf URL: https://github.com/apache/bookkeeper/issues/374 *Problem* Currently we maintain two copies for configs: one is bk_server.conf while the other one is in site/_data/bk_server.yaml for documentati

[GitHub] eolivelli commented on issue #262: Useless option "explicitLacInterval" is set to 1 in default bk_server.conf file

eolivelli commented on issue #262: Useless option "explicitLacInterval" is set to 1 in default bk_server.conf file URL: https://github.com/apache/bookkeeper/issues/262#issuecomment-320067564 I will send a PR This is an autom

[GitHub] eolivelli commented on a change in pull request #373: Issue #372: Allow to configure advertised address in bookies

eolivelli commented on a change in pull request #373: Issue #372: Allow to configure advertised address in bookies URL: https://github.com/apache/bookkeeper/pull/373#discussion_r131236461 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/Bookie.java ##

[GitHub] merlimat opened a new pull request #373: Issue #372: Allow to configure advertised address in bookies

merlimat opened a new pull request #373: Issue #372: Allow to configure advertised address in bookies URL: https://github.com/apache/bookkeeper/pull/373 Add option to configure advertised address for bookies. Reasons for this are outlined in #372 -

[GitHub] merlimat opened a new issue #372: Allow to configure a specific address to advertise the bookie

merlimat opened a new issue #372: Allow to configure a specific address to advertise the bookie URL: https://github.com/apache/bookkeeper/issues/372 Currently the bookies advertise itself, by default based on its own IP. There are ways to control: * Which IP to use, by selecting

[GitHub] sijie commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions

sijie commented on a change in pull request #342: Issue 338: add first draft Docker image including community suggestions URL: https://github.com/apache/bookkeeper/pull/342#discussion_r131218808 ## File path: docker/Makefile ## @@ -0,0 +1,197 @@ +#!/bin/bash +# +#/** +# *

[GitHub] sijie commented on issue #371: Issue 367: [documentation] add api readUnconfirmedEntries in website doc

sijie commented on issue #371: Issue 367: [documentation] add api readUnconfirmedEntries in website doc URL: https://github.com/apache/bookkeeper/pull/371#issuecomment-320046604 /cc @lucperkins for reviewing this as well. Th

[GitHub] sijie commented on issue #262: Useless option "explicitLacInterval" is set to 1 in default bk_server.conf file

sijie commented on issue #262: Useless option "explicitLacInterval" is set to 1 in default bk_server.conf file URL: https://github.com/apache/bookkeeper/issues/262#issuecomment-320046048 yes we should include this in 4.5. can you help with that? ---

Re: RackawareEnsemblePlacementPolicy only ensure at least 2 racks?

On Thu, Aug 3, 2017 at 10:31 AM, Yiming Zang wrote: > That make sense, I didn't consider the case when ensemble size > write > quorum size. So I think as you mentioned before, we can add an option to > enforce strong rack diversity. > > Considering the case of (4,2,2), I think we can't let num of

Re: RackawareEnsemblePlacementPolicy only ensure at least 2 racks?

That make sense, I didn't consider the case when ensemble size > write quorum size. So I think as you mentioned before, we can add an option to enforce strong rack diversity. Considering the case of (4,2,2), I think we can't let num of racks = min (num of racks, write quorum size). Because in this

Re: Number of ledgers and checkpoint time - bookie does not scale ?

2017-08-03 15:56 GMT+02:00 Enrico Olivelli : > > > 2017-08-02 19:21 GMT+02:00 Enrico Olivelli : > >> >> >> Il mer 2 ago 2017, 18:51 Sijie Guo ha scritto: >> >>> On Wed, Aug 2, 2017 at 8:21 AM, Enrico Olivelli >>> wrote: >>> >>> > Hi, >>> > I am trying to understand because after running my bench

[GitHub] eolivelli commented on issue #262: Useless option "explicitLacInterval" is set to 1 in default bk_server.conf file

eolivelli commented on issue #262: Useless option "explicitLacInterval" is set to 1 in default bk_server.conf file URL: https://github.com/apache/bookkeeper/issues/262#issuecomment-319983079 @sijie @zhaijack should be include this in 4.5 ?

Re: Number of ledgers and checkpoint time - bookie does not scale ?

2017-08-02 19:21 GMT+02:00 Enrico Olivelli : > > > Il mer 2 ago 2017, 18:51 Sijie Guo ha scritto: > >> On Wed, Aug 2, 2017 at 8:21 AM, Enrico Olivelli >> wrote: >> >> > Hi, >> > I am trying to understand because after running my benchmark (see [1]) >> the >> > overall performances slow down incr

[GitHub] zhaijack opened a new pull request #371: Issue 367: [documentation] add api readUnconfirmedEntries in website doc

zhaijack opened a new pull request #371: Issue 367: [documentation] add api readUnconfirmedEntries in website doc URL: https://github.com/apache/bookkeeper/pull/371 Descriptions of the changes in this PR: add api readUnconfirmedEntries in website doc --- Be sure to do all o

[GitHub] jiazhai closed pull request #369: Issue 362: [documentation] update command line tools

jiazhai closed pull request #369: Issue 362: [documentation] update command line tools URL: https://github.com/apache/bookkeeper/pull/369 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] jiazhai closed issue #362: [documentation] update command line tools

jiazhai closed issue #362: [documentation] update command line tools URL: https://github.com/apache/bookkeeper/issues/362 This is an automated message from the Apache Git Service. To respond to the message, please log on Gi

[GitHub] jiazhai closed issue #360: [documentation] configuration settings

jiazhai closed issue #360: [documentation] configuration settings URL: https://github.com/apache/bookkeeper/issues/360 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] jiazhai closed pull request #368: Issue 360: [documentation] configuration settings

jiazhai closed pull request #368: Issue 360: [documentation] configuration settings URL: https://github.com/apache/bookkeeper/pull/368 This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] zhaijack commented on issue #329: Docker image: gracefully handle start and stop bookie gracefully

zhaijack commented on issue #329: Docker image: gracefully handle start and stop bookie gracefully URL: https://github.com/apache/bookkeeper/issues/329#issuecomment-319899269 I have recorded these command into README in PR #342 . And these command could be an alternative to the default [b

Re: RackawareEnsemblePlacementPolicy only ensure at least 2 racks?

On Wed, Aug 2, 2017 at 11:34 PM, Yiming Zang wrote: > Hi @Sijie, it make sense to me to choose bookies at the same rack as many > as possible and have a least one replica at a remote rack for > availability, so that we can have lower latency within the same rack and > availability when one rack i