seatunnel icon indicating copy to clipboard operation
seatunnel copied to clipboard

[Feature][Connector-V2] connector-kafka source support data conversion extracted by kafka connect source

Open sunxiaojian opened this issue 2 years ago • 3 comments

Purpose of this pull request

#4137

Check list

  • [ ] Code changed are covered with tests, or it does not need tests for reason:
  • [ ] If any new Jar binary package adding in your PR, please add License Notice according New License Guide
  • [ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs
  • [ ] If you are contributing the connector code, please check that the following files are updated:
    1. Update change log that in connector document. For more details you can refer to connector-v2
    2. Update plugin-mapping.properties and add new connector information in it
    3. Update the pom file of seatunnel-dist
  • [ ] Update the release-note.

sunxiaojian avatar Apr 07 '23 09:04 sunxiaojian

@hailin0 @TyrantLucifer

sunxiaojian avatar Apr 11 '23 12:04 sunxiaojian

@hailin0 @TyrantLucifer

Hi @sunxiaojian , Overall LGTM, thank you for your contribution of Apache SeaTunnel, but unfortunately this current pull request overlaps in functionality with #3981, don't worry, your proposed solution and implementation approach are better. This pull request can be merged after #3981. Please be patient and wait for good news. Once there are new updates, I will ping you on this page.

TyrantLucifer avatar Apr 11 '23 14:04 TyrantLucifer

@hailin0 PTAL

EricJoy2048 avatar May 05 '23 11:05 EricJoy2048

@sunxiaojian please resolve the conflict

hailin0 avatar Jul 25 '23 03:07 hailin0

@sunxiaojian please resolve the conflict

@hailin0 done

sunxiaojian avatar Jul 25 '23 11:07 sunxiaojian

@sunxiaojian please resolve the conflict

@hailin0 done

@sunxiaojian Duplicate with https://github.com/apache/seatunnel/pull/5066 feature?

hailin0 avatar Jul 26 '23 06:07 hailin0

@sunxiaojian please resolve the conflict

@hailin0 done

@sunxiaojian Duplicate with #5066 feature?

@hailin0 It is also compatible with debezium and can be removed

sunxiaojian avatar Jul 26 '23 12:07 sunxiaojian

check ci

hailin0 avatar Jul 27 '23 02:07 hailin0

@hailin0 PTAL

sunxiaojian avatar Aug 03 '23 10:08 sunxiaojian

CI error is not related to this modification

sunxiaojian avatar Aug 09 '23 10:08 sunxiaojian