seatunnel icon indicating copy to clipboard operation
seatunnel copied to clipboard

[Fix][Connector-V2] Field information lost during Paimon DataType and SeaTunnel Column conversion

Open xiaochen-zhou opened this issue 1 year ago • 4 comments

Purpose of this pull request

Field information lost during Paimon DataType and SeaTunnel Column conversion, such as: field name

Does this PR introduce any user-facing change? no

How was this patch tested? exists test

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.

xiaochen-zhou avatar Apr 28 '24 05:04 xiaochen-zhou

Any test case?

Hisoka-X avatar Apr 28 '24 07:04 Hisoka-X

Any test case?

Any test case?

OK, thanks for your review. I'm going to add some test cases

xiaochen-zhou avatar Apr 29 '24 06:04 xiaochen-zhou

@xiaochen-zhou Please resolve the conflicts.

dailai avatar May 22 '24 02:05 dailai

@xiaochen-zhou Please resolve the conflicts.

image

dailai avatar May 22 '24 07:05 dailai

@xiaochen-zhou Please resolve the conflicts.

image

done.

xiaochen-zhou avatar May 22 '24 12:05 xiaochen-zhou

LGTM if ci passed.

done.

xiaochen-zhou avatar May 22 '24 23:05 xiaochen-zhou