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

   ### Motivation
   
   When I try to learn the transaction part features. I found the 
``TransactionMetadataStoreService#endTransaction`` method to look complicated. 
I think I should try to refactor it to reduce the complexity caused by many 
``CompletableFuture``.
   
   ### Modifications
   
   - Use chain-call to avoid catching exceptions everywhere.
   
   - Try to avoid using multiple CompletableFutures to increase understanding 
complexity.
   
   ### Verifying this change
   
   - [x] Make sure that the change passes the CI checks.
   
   ### Documentation
   
   - [x] `no-need-doc` 


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