close
Skip to content

[Improve][Connector-V2] Support hive catalog for paimon sink#6833

Merged
hailin0 merged 2 commits into
apache:devfrom
dailai:support-hivecatalog-for-paimonsink
May 22, 2024
Merged

[Improve][Connector-V2] Support hive catalog for paimon sink#6833
hailin0 merged 2 commits into
apache:devfrom
dailai:support-hivecatalog-for-paimonsink

Conversation

@dailai

@dailai dailai commented May 11, 2024 •
edited by hailin0
Loading

Copy link
Copy Markdown
Contributor

Purpose of this pull request

Support hive catalog for paimon sink. close #6821

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

@dailai

dailai commented May 11, 2024

Copy link
Copy Markdown
Contributor Author

截屏2024-05-11 10 37 02
截屏2024-05-11 10 37 32
截屏2024-05-11 10 38 00
截屏2024-05-11 10 38 30

@dailai dailai force-pushed the support-hivecatalog-for-paimonsink branch from 06e2a0e to 619327a Compare May 11, 2024 06:24
@hailin0 hailin0 added this to the 2.3.6 milestone May 13, 2024
@dailai dailai force-pushed the support-hivecatalog-for-paimonsink branch from 619327a to fcd3814 Compare May 15, 2024 03:17
EricJoy2048
EricJoy2048 reviewed May 15, 2024
View reviewed changes
Comment thread ...n/src/main/java/org/apache/seatunnel/connectors/seatunnel/paimon/sink/PaimonSinkFactory.java Outdated
Comment thread seatunnel-common/src/main/java/org/apache/seatunnel/common/exception/CommonError.java
Comment thread ...main/java/org/apache/seatunnel/connectors/seatunnel/paimon/catalog/PaimonCatalogFactory.java Outdated
Comment thread ...imon/src/main/java/org/apache/seatunnel/connectors/seatunnel/paimon/config/PaimonConfig.java Outdated
Comment thread ...tor-paimon-e2e/src/test/java/org/apache/seatunnel/e2e/connector/paimon/PaimonSinkHdfsIT.java
@dailai dailai requested a review from EricJoy2048 May 16, 2024 05:50
@dailai dailai force-pushed the support-hivecatalog-for-paimonsink branch from 585e039 to a9d87f3 Compare May 21, 2024 02:59
Carl-Zhou-CN
Carl-Zhou-CN previously approved these changes May 21, 2024
View reviewed changes

@Carl-Zhou-CN Carl-Zhou-CN left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM If Ci passes

@dailai dailai force-pushed the support-hivecatalog-for-paimonsink branch from 31a4a01 to 5c36d23 Compare May 21, 2024 06:44
@dailai dailai requested a review from Carl-Zhou-CN May 21, 2024 06:45
@dailai dailai force-pushed the support-hivecatalog-for-paimonsink branch from 5c36d23 to 5cde271 Compare May 21, 2024 07:02
Carl-Zhou-CN
Carl-Zhou-CN approved these changes May 21, 2024
View reviewed changes

@Carl-Zhou-CN Carl-Zhou-CN left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

hailin0
hailin0 approved these changes May 22, 2024
View reviewed changes

@hailin0 hailin0 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hailin0 hailin0 merged commit 4969c91 into apache:dev May 22, 2024
@dailai dailai deleted the support-hivecatalog-for-paimonsink branch June 13, 2024 08:11
chaorongzhi pushed a commit to chaorongzhi/seatunnel that referenced this pull request Aug 21, 2024
@dailai
[Improve][Connector-V2] Support hive catalog for paimon sink (apache#…
fb19282 
…6833)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hailin0 hailin0 hailin0 approved these changes
@Carl-Zhou-CN Carl-Zhou-CN Carl-Zhou-CN approved these changes
@EricJoy2048 EricJoy2048 Awaiting requested review from EricJoy2048

Assignees

No one assigned

Labels

approved improve

Projects

None yet

Milestone

2.3.6

Development

Successfully merging this pull request may close these issues.

[Feature][Connector-V2] Support Hive catalog for paimon sink

4 participants

@dailai @hailin0 @EricJoy2048 @Carl-Zhou-CN