You are viewing a plain text version of this content. The canonical link for it is here.

Posted to jira@kafka.apache.org by GitBox <gi...@apache.org> on 2021/02/25 03:32:13 UTC

[GitHub] [kafka] twmb edited a comment on pull request #10183: KAFKA-12267; Implement `DescribeTransactions` API

twmb edited a comment on pull request #10183:
URL: https://github.com/apache/kafka/pull/10183#issuecomment-785548579


   Can the KIP be updated to change TransactionState from an int8 to a string (as implemented in this PR)?
   
   Alternatively, is there value in using a string vs. an int8? The string is more descriptive, but since the states are a defined enum, it'd be smaller to serialize/deserialize the int8 and then a client can perform the int8 => string conversion.


----------------------------------------------------------------
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:
users@infra.apache.org