liubingxing commented on PR #4149:
URL: https://github.com/apache/bookkeeper/pull/4149#issuecomment-1958775974

   > The test failed with timeout
   > 
   > 
https://github.com/apache/bookkeeper/blob/a79a4e21e838d595e63b9e62b53951dd34a89c66/tools/ledger/src/test/java/org/apache/bookkeeper/tools/cli/commands/bookie/ReadLedgerCommandTest.java#L131-L140
   > 
   > The failed log is
   > 
   > ```
   > 2023-12-11T06:54:35.6468730Z [INFO] Running 
org.apache.bookkeeper.tools.cli.commands.bookie.LastMarkCommandTest
   > 2023-12-11T06:54:37.4352864Z [INFO] Tests run: 1, Failures: 0, Errors: 0, 
Skipped: 0, Time elapsed: 1.787 s - in 
org.apache.bookkeeper.tools.cli.commands.bookie.LastMarkCommandTest
   > 2023-12-11T06:54:37.6435604Z [INFO] Running 
org.apache.bookkeeper.tools.cli.commands.bookie.ReadLedgerCommandTest
   > 2023-12-11T07:24:37.9277456Z [INFO] Running 
org.apache.bookkeeper.tools.cli.commands.bookie.SanityTestCommandTest
   > 2023-12-11T07:24:39.8057608Z [INFO] Tests run: 5, Failures: 0, Errors: 0, 
Skipped: 0, Time elapsed: 1.876 s - in 
org.apache.bookkeeper.tools.cli.commands.bookie.SanityTestCommandTest
   > 2023-12-11T07:24:40.0306262Z [INFO] Running 
org.apache.bookkeeper.tools.cli.commands.bookie.ReadLogCommandTest
   > ```
   > 
   > @liubingxing Please take a look, thanks.
   
   @hangc0276 Thanks for pointing out the error, I updated the code. Please 
review again.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to