ijuma commented on a change in pull request #10573:
URL: https://github.com/apache/kafka/pull/10573#discussion_r629848073
##########
File path:
clients/src/main/java/org/apache/kafka/clients/producer/KafkaProducer.java
##########
@@ -525,6 +525,11 @@ private TransactionManager
configureTransactionState(ProducerConfig config,
final int transactionTimeoutMs =
config.getInt(ProducerConfig.TRANSACTION_TIMEOUT_CONFIG);
final long retryBackoffMs =
config.getLong(ProducerConfig.RETRY_BACKOFF_MS_CONFIG);
final boolean autoDowngradeTxnCommit =
config.getBoolean(ProducerConfig.AUTO_DOWNGRADE_TXN_COMMIT);
+ // Only log a warning if being used outside of Streams, which we
know includes "StreamThread-" in the client id
+ if (autoDowngradeTxnCommit && !clientId.contains("StreamThread-"))
{
Review comment:
4.0 is probably 18 months away, that's a reasonably long time. Why do we
need to log a warning only if it's not Streams?
--
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
