[ 
https://issues.apache.org/jira/browse/KAFKA-4327?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16150259#comment-16150259
 ] 

Jorge Quilcate commented on KAFKA-4327:
---------------------------------------

Thanks [~ijuma], I will move this tool to the `streams` module.

Based on 
[KIP-171](https://cwiki.apache.org/confluence/display/KAFKA/KIP-171+-+Extend+Consumer+Group+Reset+Offset+for+Stream+Application),
 to implement changes on this tool now we will need to call 
`ConsumerGroupCommand` to reset offsets. This would require a dependency to 
`core_main` as far as I understand. Would this be an issue?

I have a naive implementation of KIP-171 here: 
https://github.com/jeqo/kafka/commits/features/kip-171

> Move Reset Tool from core to streams
> ------------------------------------
>
>                 Key: KAFKA-4327
>                 URL: https://issues.apache.org/jira/browse/KAFKA-4327
>             Project: Kafka
>          Issue Type: Improvement
>          Components: streams
>            Reporter: Matthias J. Sax
>            Assignee: Jorge Quilcate
>            Priority: Minor
>
> This is a follow up on https://issues.apache.org/jira/browse/KAFKA-4008
> Currently, Kafka Streams Application Reset Tool is part of {{core}} module 
> due to ZK dependency. After KIP-4 got merged, this dependency can be dropped 
> and the Reset Tool can be moved to {{streams}} module.
> This should also update {{InternalTopicManager#filterExistingTopics}} that 
> revers to ResetTool in an exception message:
> {{"Use 'kafka.tools.StreamsResetter' tool"}}
> -> {{"Use '" + kafka.tools.StreamsResetter.getClass().getName() + "' tool"}}
> Doing this JIRA also requires to update the docs with regard to broker 
> backward compatibility -- not all broker support "topic delete request" and 
> thus, the reset tool will not be backward compatible to all broker versions.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to