Re: [PR] [FLINK-32743][Connectors/Kafka] Parse data from kafka connect and convert it into regular JSON data [flink-connector-kafka]

2024-02-15 Thread via GitHub


mas-chen commented on PR #42:
URL: 
https://github.com/apache/flink-connector-kafka/pull/42#issuecomment-1947817531

   @sunxiaojian have you considered implementing this as a Flink format? You 
could draw inspiration from pre-existing formats: 
https://github.com/apache/flink/tree/master/flink-formats/flink-json/src/main/java/org/apache/flink/formats/json


-- 
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: issues-unsubscr...@flink.apache.org

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



Re: [PR] [FLINK-32743][Connectors/Kafka] Parse data from kafka connect and convert it into regular JSON data [flink-connector-kafka]

2024-01-29 Thread via GitHub


MartijnVisser commented on PR #42:
URL: 
https://github.com/apache/flink-connector-kafka/pull/42#issuecomment-1914342095

   @mas-chen Want to take a look?


-- 
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: issues-unsubscr...@flink.apache.org

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



Re: [PR] [FLINK-32743][Connectors/Kafka] Parse data from kafka connect and convert it into regular JSON data [flink-connector-kafka]

2024-01-25 Thread via GitHub


sunxiaojian commented on PR #42:
URL: 
https://github.com/apache/flink-connector-kafka/pull/42#issuecomment-1911592095

   > > Thanks for your review, it has been processed
   > 
   > CI is failing, will you take a look?
   
   @MartijnVisser fixed


-- 
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: issues-unsubscr...@flink.apache.org

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



Re: [PR] [FLINK-32743][Connectors/Kafka] Parse data from kafka connect and convert it into regular JSON data [flink-connector-kafka]

2024-01-25 Thread via GitHub


MartijnVisser commented on PR #42:
URL: 
https://github.com/apache/flink-connector-kafka/pull/42#issuecomment-1911093338

   > Thanks for your review, it has been processed
   
   CI is failing, will you take a look?


-- 
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: issues-unsubscr...@flink.apache.org

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



Re: [PR] [FLINK-32743][Connectors/Kafka] Parse data from kafka connect and convert it into regular JSON data [flink-connector-kafka]

2024-01-18 Thread via GitHub


sunxiaojian commented on PR #42:
URL: 
https://github.com/apache/flink-connector-kafka/pull/42#issuecomment-1899549561

   > @sunxiaojian Can you please rebase your PR?
   
   @MartijnVisser Thanks for your review, it has been processed


-- 
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: issues-unsubscr...@flink.apache.org

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



Re: [PR] [FLINK-32743][Connectors/Kafka] Parse data from kafka connect and convert it into regular JSON data [flink-connector-kafka]

2024-01-18 Thread via GitHub


MartijnVisser commented on PR #42:
URL: 
https://github.com/apache/flink-connector-kafka/pull/42#issuecomment-1898688209

   @sunxiaojian Can you please rebase your PR?


-- 
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: issues-unsubscr...@flink.apache.org

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