[bookkeeper] 03/06: ISSUE #274: Ensuring the right number of bookies are passed.

[si...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

sijie pushed a commit to branch branch-4.5
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git

commit 61bcba2194aa28ebd51c83e0448965c18211319a
Author: arvindkandhare <ar...@emc.com>
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 <ar...@emc.com>
    
    Reviewers: Enrico Olivelli <eo...@gmail.com>, Jia Zhai <None>, Sijie Guo <si...@apache.org>
    
    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
"commits@bookkeeper.apache.org" <co...@bookkeeper.apache.org>.