This is an automated email from the ASF dual-hosted git repository.
eolivelli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git
The following commit(s) were added to refs/heads/master by this push:
new 29e7422 ISSUE #274: Ensuring the right number of bookies are passed.
29e7422 is described below
commit 29e74229eda138c13c88ec890ce3d2c215972303
Author: arvindkandhare <[email protected]>
AuthorDate: Wed Aug 30 09:30:31 2017 +0200
ISSUE #274: Ensuring the right number of bookies are passed.
Descriptions of the changes in this PR:
This fixes #274. Ensures that benchmark measure right number of bookies for
the warmup.
Author: arvindkandhare <[email protected]>
Reviewers: Enrico Olivelli <[email protected]>, Jia Zhai <None>, Sijie
Guo <[email protected]>
This closes #482 from arvindkandhare/issue-274-benchmarkfix, closes #274
---
.../java/org/apache/bookkeeper/benchmark/BenchThroughputLatency.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/bookkeeper-benchmark/src/main/java/org/apache/bookkeeper/benchmark/BenchThroughputLatency.java
b/bookkeeper-benchmark/src/main/java/org/apache/bookkeeper/benchmark/BenchThroughputLatency.java
index 483b002..49febbe 100644
---
a/bookkeeper-benchmark/src/main/java/org/apache/bookkeeper/benchmark/BenchThroughputLatency.java
+++
b/bookkeeper-benchmark/src/main/java/org/apache/bookkeeper/benchmark/BenchThroughputLatency.java
@@ -441,7 +441,7 @@ public class BenchThroughputLatency implements AddCallback,
Runnable {
LOG.error("Couldn't connect to zookeeper at " + servers);
throw new IOException("Couldn't connect to zookeeper " +
servers);
}
- bookies = zk.getChildren(bookieRegistrationPath, false).size();
+ bookies = zk.getChildren(bookieRegistrationPath, false).size() - 1;
} finally {
if (zk != null) {
zk.close();
--
To stop receiving notification emails like this one, please contact
['"[email protected]" <[email protected]>'].
