sijie commented on a change in pull request #1042: ISSUE #1041: Adding new
testcases
URL: https://github.com/apache/bookkeeper/pull/1042#discussion_r163145442
##########
File path:
bookkeeper-server/src/test/java/org/apache/bookkeeper/client/BookieWriteLedgerTest.java
##########
@@ -344,6 +347,88 @@ public void testLedgerCreateWithCustomMetadata() throws
Exception {
}
/*
+ * Verify the functionality of Advanced Ledger which accepts ledgerId as
+ * input and returns LedgerHandleAdv. LedgerHandleAdv takes entryId for
+ * addEntry, and let user manage entryId allocation.
+ * This testcase is mainly added for covering missing code coverage
branches
+ * in LedgerHandleAdv
+ *
+ * @throws Exception
+ */
+ @Test(timeout = 60000)
Review comment:
nit: remove timeout
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
