[
https://issues.apache.org/jira/browse/KAFKA-17274?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17871404#comment-17871404
]
Chia-Ping Tsai commented on KAFKA-17274:
----------------------------------------
I set fix version to 5.0.0, as normally we need to follow one-year deprecation
cycle.
> Remove deprecate `TransactionLogMessageFormatter`,
> GroupMetadataMessageFormatter` and `OffsetsMessageFormatter`
> ---------------------------------------------------------------------------------------------------------------
>
> Key: KAFKA-17274
> URL: https://issues.apache.org/jira/browse/KAFKA-17274
> Project: Kafka
> Issue Type: Improvement
> Components: core
> Affects Versions: 4.0.0
> Reporter: 黃竣陽
> Assignee: 黃竣陽
> Priority: Minor
> Fix For: 5.0.0
>
>
> In KAFKA-16666
> we already migrate `GroupMetadataMessageFormatter`[0],
> `OffsetsMessageFormatter`[1], and `TransactionLogMessageFormatter`[2] , and
> deprecate for these formatters, thus at the next major version, we should
> remove it.
> Also we should remove ConsoleConsumerOptions#convertDeprecatedClass[3] switch
> case statement, and
> ConsoleConsumerOptionsTest#testNewAndDeprecateTransactionLogMessageFormatter[4],
>
> testNewAndDeprecateOffsetsMessageFormatter,
> testNewAndDeprecateGroupMetadataMessageFormatter
> [0]
> [https://github.com/apache/kafka/blob/trunk/core/src/main/scala/kafka/coordinator/group/GroupMetadataManager.scala#L1261]
> [1]
> [https://github.com/apache/kafka/blob/trunk/core/src/main/scala/kafka/coordinator/group/GroupMetadataManager.scala#L1239]
> [2]
> [https://github.com/apache/kafka/blob/trunk/core/src/main/scala/kafka/coordinator/transaction/TransactionLog.scala#L150]
> [3]
> [https://github.com/apache/kafka/blob/trunk/tools/src/main/java/org/apache/kafka/tools/consumer/ConsoleConsumerOptions.java#L353]
> [4]
> [https://github.com/apache/kafka/blob/trunk/tools/src/test/java/org/apache/kafka/tools/consumer/ConsoleConsumerOptionsTest.java#L664]
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
