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

Richard Joerger commented on KAFKA-14326:
-----------------------------------------

Hi Team, I would be happy to work on this. I think the code change would be 
converting the array of values to a stream and then mapping trim() for each 
value. A couple questions to help whomever gets started on this (including 
myself):
1) Are there any tests which exist for testing the tool itself? It doesn't 
appear to be but I may just be looking in the wrong place. 
2) Since any changes made to this would effect public facing tooling, should a 
KIP be written up?
3) Would it make more sense to open an issue focusing on documentation of this 
or just include any changes to that in the PR?


> kafka-application-reset-tool should accept spaces between input/intermediate 
> topics
> -----------------------------------------------------------------------------------
>
>                 Key: KAFKA-14326
>                 URL: https://issues.apache.org/jira/browse/KAFKA-14326
>             Project: Kafka
>          Issue Type: Improvement
>          Components: streams
>    Affects Versions: 3.0.1
>            Reporter: Matt Allwood
>            Priority: Minor
>
> When running the application reset tool, the tool doesn't recognise topics 
> when there is whitespace separating them. This can affect readability and 
> maintainability of stored scripts where we may want to have each topic on a 
> new line, unless we use string concatenation
> It would be better if the tool would strip whitespace from around topic names



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to