seatunnel icon indicating copy to clipboard operation
seatunnel copied to clipboard

[Feature][SQL Config] Add SQL config adapter

Open rewerma opened this issue 1 year ago • 2 comments

Purpose of this pull request

[Feature][SQL Config] Add SQL config adapter

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

  • [ ] 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/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.

rewerma avatar Apr 25 '24 08:04 rewerma

cc @EricJoy2048 @Hisoka-X

hailin0 avatar May 06 '24 04:05 hailin0

Could you add some unit test for the new module named seatunnel-config-sql to vertify the logical? I think you can add some case like that the password in options is auto generate like this: ^%QjiXCG&GR'ob-T8Lr6UNaqQC#;7).

image

dailai avatar May 08 '24 05:05 dailai