You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2017/11/02 11:05:52 UTC

[GitHub] ivankelly opened a new pull request #684: ISSUE #683: Correctly update length in LedgerHandleAdv

ivankelly opened a new pull request #684: ISSUE #683: Correctly update length in LedgerHandleAdv
URL: https://github.com/apache/bookkeeper/pull/684
 
 
   Commit 7b1eec introduced a change which removed the length parameter
   to doAsyncAddEntry. This resulted in the length of the ledger being
   added to itself each time for a LedgerHandleAdv, which ultimately
   meant that that length would be 0, as it never had anything non-zero
   added to it.
   
   This change corrects this, by adding the length of the data parameter
   to the ledger length.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services