[Improve][Paimon] upgrade version and add fs config

apache / seatunnel

SeaTunnel is a next-generation super high-performance, distributed, massive data integration tool.

https://seatunnel.apache.org/

Apache License 2.0

8.06k stars 1.83k forks source link

[Improve][Paimon] upgrade version and add fs config #8002

Closed yoogoc closed 1 week ago

yoogoc commented 2 weeks ago

Purpose of this pull request

Does this PR introduce any user-facing change?

How was this patch tested?

debug in local

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 plugin-mapping.properties and add new connector information in it
2. Update the pom file of seatunnel-dist
3. Add ci label in label-scope-conf
4. Add e2e testcase in seatunnel-e2e
5. Update connector plugin_config
[ ] Update the release-note.

Hisoka-X commented 1 week ago

Hi @yoogoc , please follow the guide to open github action on your fork repository. https://github.com/apache/seatunnel/pull/8002/checks?check_run_id=32718704961

yoogoc commented 1 week ago

paimon 0.9.0 have a bug about handle tinyint and smallint correctly bug(pr here). I'm not sure whether to skip e2e or wait for the new version. @Hisoka-X

Hisoka-X commented 1 week ago

paimon 0.9.0 have a bug about handle tinyint and smallint correctly bug(pr here). I'm not sure whether to skip e2e or wait for the new version. @Hisoka-X

Thanks @yoogoc for share this information. I think we should wait for 0.9.1. Any update can not break feature we already supported.

hawk9821 commented 1 week ago

Thank you @yoogoc A PR should only do one thing, and upgrades and fs support should be separate.

dailai commented 1 week ago

Thank you @yoogoc A PR should only do one thing, and upgrades and fs support should be separate.

+1 It is recommended that upgrading the paimon version be a separate pr, which is a big deal.

hawk9821 commented 1 week ago

Thanks @yoogoc, I already support upgrade Piamon version to 0.8.2 , to be compatible with doris ecology. doris-paimon

yoogoc commented 1 week ago

thanks, i close the pr now

yoogoc commented 1 week ago

Thanks @yoogoc, I already support upgrade Piamon version to 0.8.2 , to be compatible with doris ecology. doris-paimon

cool