[GitHub] [kafka] jolshan commented on pull request #13231: KAFKA-14402: Update AddPartitionsToTxn protocol to batch and handle verifyOnly requests

["jolshan (via GitHub)" <gi...@apache.org>]

jolshan commented on PR #13231:
URL: https://github.com/apache/kafka/pull/13231#issuecomment-1441071519

   I've added the changes to the API spec
   -- verify only is now a transaction level config
   -- top level error is added to the response
   
   I've added builders to the request and tried to simplify some of the methods.
   I've also addressed the verifyOnly case where some partitions are in the txn and others are not. 
   
   I will update the KIP to reflect some of these changes (especially with respect to the API spec)
   
   I still need to address the unstable API change, but that will require a pull from master.


-- 
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: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org