close
Skip to content

[Hotfix][Connector-V2][Hive] fix the bug that hive-site.xml can not be injected#5261

Merged
EricJoy2048 merged 1 commit into
apache:devfrom
VolodymyrDuke:fix-hive-conf
Sep 11, 2023
Merged

[Hotfix][Connector-V2][Hive] fix the bug that hive-site.xml can not be injected#5261
EricJoy2048 merged 1 commit into
apache:devfrom
VolodymyrDuke:fix-hive-conf

Conversation

@VolodymyrDuke

Copy link
Copy Markdown
Contributor

Purpose of this pull request

Check list

@VolodymyrDuke

Copy link
Copy Markdown
Contributor Author

@TyrantLucifer @hailin0 @EricJoy2048 PTAL.

String.format(
"Using this hive uris [%s], hive conf [%s] to initialize "
+ "hive metastore client instance failed",
metastoreUri, config.getString(HiveConfig.HIVE_SITE_PATH.key()));

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.

When we reach here, the HiveConfig.HIVE_SITE_PATH must not be null?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Still got the error :
org.apache.flink.client.program.ProgramInvocationException: The main method caused an error: ErrorCode:[HIVE-03], ErrorDescription:[Get hive table information from hive metastore service failed] - Get table [ods_wj.order_analysis] information failed
Caused by: org.apache.thrift.transport.TTransportException

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

When we reach here, the HiveConfig.HIVE_SITE_PATH must not be null?

yes, you are right, I will check it. Thank you!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It has no problem. Thank you for your review. @Hisoka-X

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.

Does @SinyoWong reported error is related to this PR?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Does @SinyoWong reported error is related to this PR?

Oh! I mean this issue:
#4451

@Hisoka-X Hisoka-X added this to the 2.3.4 milestone Aug 22, 2023
@EricJoy2048 EricJoy2048 merged commit 04ce22a into apache:dev Sep 11, 2023
Zhouwen-CN pushed a commit to Zhouwen-CN/seatunnel that referenced this pull request Sep 11, 2023
Zhouwen-CN pushed a commit to Zhouwen-CN/seatunnel that referenced this pull request Sep 11, 2023
Zhouwen-CN pushed a commit to Zhouwen-CN/seatunnel that referenced this pull request Sep 11, 2023
gnehil pushed a commit to gnehil/seatunnel that referenced this pull request Oct 12, 2023
@FengSq

FengSq commented Feb 21, 2024

Copy link
Copy Markdown

为什么还报这个问题啊
seatunnel 2.3.4
hive 3.x
hive-exec.2.3.9

@zhaoli2333

Copy link
Copy Markdown

@FengSq 解决了吗?
hive 2.x hive-exec 2.3.9 一样报错

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] [Connector-hive-v2] can't get table from hive3.x by the seatunnel 2.3.1-release

7 participants