nicoloboschi opened a new pull request, #19137:
URL: https://github.com/apache/pulsar/pull/19137

   ### Motivation
   
   When the broker sends the transaction end response which has failed, the 
least sig bits is not included in the response payload. 
   This is not a big deal since the java client only uses it for debug logging.
   
   However, thanks to #14613, it's possible to intercept this response payload 
so it's better to pass the correct value.   
   
   ### Modifications
   
   * Set `TxnidLeastBits` in `CommandEndTxnResponse`
   
   ### Verifying this change
   
   - [x] Make sure that the change passes the CI checks.
   
   This change is a trivial rework / code cleanup without any test coverage.
   ### Documentation
   
   <!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
   
   - [ ] `doc` <!-- Your PR contains doc changes. Please attach the local 
preview screenshots (run `sh start.sh` at `pulsar/site2/website`) to your PR 
description, or else your PR might not get merged. -->
   - [ ] `doc-required` <!-- Your PR changes impact docs and you will update 
later -->
   - [x] `doc-not-needed` <!-- Your PR changes do not impact docs -->
   - [ ] `doc-complete` <!-- Docs have been already added -->
   


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