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/03/01 21:15:59 UTC

[GitHub] [kafka] vanhoale opened a new pull request #10239: KAFKA-12372 - Enhance TimestampCoverter to handle multiple timestamp or date fields

vanhoale opened a new pull request #10239:
URL: https://github.com/apache/kafka/pull/10239


   [JIRA](https://issues.apache.org/jira/browse/KAFKA-12372)
   
   Our team is having an issue of handling multiple timestamp fields in a kafka message, so for now if we use the converter then we have to add more fields in the config.
   
   We can implement it in a generic way to check if the field.schema().name() is timestamp or date then we can convert it.


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

[GitHub] [kafka] vanhoale removed a comment on pull request #10239: KAFKA-12372: Enhance TimestampCoverter Connect transformation to handle multiple timestamp or date fields

Posted by GitBox <gi...@apache.org>.
vanhoale removed a comment on pull request #10239:
URL: https://github.com/apache/kafka/pull/10239#issuecomment-802059907


   hi @kkonstantine, should one of your team members review the PR? we want to use this feature ASAP 


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

[GitHub] [kafka] vanhoale commented on pull request #10239: KAFKA-12372: Enhance TimestampCoverter Connect transformation to handle multiple timestamp or date fields

Posted by GitBox <gi...@apache.org>.
vanhoale commented on pull request #10239:
URL: https://github.com/apache/kafka/pull/10239#issuecomment-802059907


   hi @kkonstantine, should one of your team members review the PR? we want to use this feature ASAP 


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

[GitHub] [kafka] vanhoale commented on pull request #10239: KAFKA-12372: Enhance TimestampCoverter Connect transformation to handle multiple timestamp or date fields

Posted by GitBox <gi...@apache.org>.
vanhoale commented on pull request #10239:
URL: https://github.com/apache/kafka/pull/10239#issuecomment-789110876


   I ran my test, checkstyle and spotbugs successful at my local, but not sure some checks failed here


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

[GitHub] [kafka] vanhoale removed a comment on pull request #10239: KAFKA-12372: Enhance TimestampCoverter Connect transformation to handle multiple timestamp or date fields

Posted by GitBox <gi...@apache.org>.
vanhoale removed a comment on pull request #10239:
URL: https://github.com/apache/kafka/pull/10239#issuecomment-789110876


   I ran my test, checkstyle and spotbugs successful at my local, but not sure some checks failed here


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