rreddy-22 commented on code in PR #19751:
URL: https://github.com/apache/kafka/pull/19751#discussion_r2096560820
##########
core/src/main/scala/kafka/coordinator/transaction/TransactionCoordinator.scala:
##########
@@ -135,10 +136,10 @@ class TransactionCoordinator(txnConfig: TransactionConfig,
// 2PC functionality is disabled, clients that attempt to use this
functionality
// would receive an authorization failed error.
responseCallback(initTransactionError(Errors.TRANSACTIONAL_ID_AUTHORIZATION_FAILED))
- } else if (keepPreparedTxn) {
- // if the request is to keep the prepared transaction, then return an
- // unsupported version error since the feature hasn't been implemented
yet.
- responseCallback(initTransactionError(Errors.UNSUPPORTED_VERSION))
+ } else if (keepPreparedTxn && expectedProducerIdAndEpoch.nonEmpty) {
+ // keepPreparedTxn can only be specified in the initial call of
InitProducerId, so
+ // no exposed producerId / epoch should be set
Review Comment:
nit: did we mean to say expected or exposed?
--
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]
