[GitHub] [kafka] rhauch edited a comment on pull request #9492: KAFKA-10627: Added support for Connect TimestampConverter to convert multiple fields using a comma-separated list, and multiple input formats when parsing from a string.

[GitBox <gi...@apache.org>]

rhauch edited a comment on pull request #9492:
URL: https://github.com/apache/kafka/pull/9492#issuecomment-888303589


   Thanks for the contribution, and thanks for the KIP: https://cwiki.apache.org/confluence/display/KAFKA/KIP-682%3A+Connect+TimestampConverter+support+for+multiple+fields+and+multiple+input+formats. I'll take a look
   
   Overall, I think the design is good as long as the SMT has the same behavior with existing configurations. But I'll review the KIP and provide feedback on the KIP discussion thread.
   
   BTW, one of the benefits of Connect SMTs is that you can develop your own implementations, package them as a custom plugin, and use them in your own Connect installations. 


-- 
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.

To unsubscribe, e-mail: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org