AnonHxy opened a new pull request, #3740:
URL: https://github.com/apache/bookkeeper/pull/3740

   Descriptions of the changes in this PR:
   
   
   
   ### Motivation
   
   The `entryId` of `ReadRequest` should be -1 instead of 0 when call 
`PerChannelBookieClient#readLac`. 
   Luckly we will call `PerChannelBookieClient#readLac` only when 
`useV2WireProtocol` is false,  so it will not cause problem in production 
environment.  But it's a potential bug if we refactor the readLac in v2 
protocol, so it's necessary to fix it I think
   
   
   
   ### Changes
   Changing `entryId` of `ReadRequest` in `PerChannelBookieClient#readLac` from 
0 to `-1`
   


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