-
Notifications
You must be signed in to change notification settings - Fork 126
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FLINK-32743][Connectors/Kafka] Parse data from kafka connect and convert it into regular JSON data #42
base: main
Are you sure you want to change the base?
Conversation
Thanks for opening this pull request! Please check out our contributing guidelines. (https://flink.apache.org/contributing/how-to-contribute.html) |
de74496
to
6d3dbe7
Compare
@sunxiaojian Can you please rebase your PR? |
6d3dbe7
to
88048d0
Compare
@MartijnVisser Thanks for your review, it has been processed |
8aafc6e
to
451a673
Compare
CI is failing, will you take a look? |
966a84d
to
cbf7725
Compare
…vert it into regular JSON data
79e43b3
to
4df2bd8
Compare
@MartijnVisser fixed |
@mas-chen Want to take a look? |
@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 For example, the debezium json format already has an integration with Kafka connect. |
@mas-chen ok, that's a good suggestion |
Add "record.key.include.kafka.connect.json.schema" and "record.value.include.kafka.connect.json.schema" configurations. When this configuration is true, records can be converted into regular JSON data