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

sijie 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 9e4bd8e  BookKeeperBuilderTest has recovery backwards
9e4bd8e is described below

commit 9e4bd8e3de91cafaf87386a29d09f3d62909a5d4
Author: Ivan Kelly <[email protected]>
AuthorDate: Wed Dec 20 18:10:28 2017 -0800

    BookKeeperBuilderTest has recovery backwards
    
    NoRecovery test was specifying withRecovery(true).
    Recovery test was specifying withRecovery(false).
    
    This patch swaps them to how they should be.
    
    Author: Ivan Kelly <[email protected]>
    
    Reviewers: Enrico Olivelli <[email protected]>, Sijie Guo 
<[email protected]>
    
    This closes #899 from ivankelly/recovery-test-fix
---
 .../java/org/apache/bookkeeper/client/api/BookKeeperBuildersTest.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/bookkeeper-server/src/test/java/org/apache/bookkeeper/client/api/BookKeeperBuildersTest.java
 
b/bookkeeper-server/src/test/java/org/apache/bookkeeper/client/api/BookKeeperBuildersTest.java
index 3c29895..f6b8b07 100644
--- 
a/bookkeeper-server/src/test/java/org/apache/bookkeeper/client/api/BookKeeperBuildersTest.java
+++ 
b/bookkeeper-server/src/test/java/org/apache/bookkeeper/client/api/BookKeeperBuildersTest.java
@@ -349,7 +349,7 @@ public class BookKeeperBuildersTest extends 
MockBookKeeperTestCase {
             .withPassword(ledgerMetadata.getPassword())
             .withDigestType(DigestType.CRC32)
             .withLedgerId(ledgerId)
-            .withRecovery(true)
+            .withRecovery(false)
             .execute());
     }
 
@@ -369,7 +369,7 @@ public class BookKeeperBuildersTest extends 
MockBookKeeperTestCase {
             .withPassword(ledgerMetadata.getPassword())
             .withDigestType(DigestType.CRC32)
             .withLedgerId(ledgerId)
-            .withRecovery(false)
+            .withRecovery(true)
             .execute());
     }
 

-- 
To stop receiving notification emails like this one, please contact
['"[email protected]" <[email protected]>'].

Reply via email to