[GitHub] sijie commented on issue #1411: LedgerEntry#getLength does not do what the documentation says

[GitBox <gi...@apache.org>]

sijie commented on issue #1411: LedgerEntry#getLength does not do what the documentation says
URL: https://github.com/apache/bookkeeper/issues/1411#issuecomment-389916787
 
 
   for BC consideration, we can fix the documentation first. 
   
   We can consider deprecating `getLength` later, introducing `getEntrySize` to return the size of the entry and `getOffset` to return the `offset` of the entry in a ledger. `entrySize + offset` == `getLength` here.
   
   
   `Offset` would be aligning with the proposal in #1376 

----------------------------------------------------------------
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