lhotari commented on PR #3675: URL: https://github.com/apache/bookkeeper/pull/3675#issuecomment-1933485215
I'm fixing the integration tests in #4197. It seems that simpletest will always fail and has never been working. There are at least 2 problems: - the last entry id parameter to the read method is invalid (off by one) - the LAC is updated asynchronously while the ledger is open. The read could fail if the entries aren't yet confirmed. Fixing in https://github.com/lhotari/bookkeeper/commit/6ffda5cb since I cannot get integration tests to pass without it. This is a sign that integration tests have been broken for a very long time on apache/bookkeeper. . The request to read the entries is -- 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]
