Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 1 commit into from
Sep 11, 2023

Conversation

VolodymyrDuke
Copy link
Contributor

Purpose of this pull request

Check list

@VolodymyrDuke
Copy link
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
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?

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
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
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
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?

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
Copy link

FengSq commented Feb 21, 2024

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

@zhaoli2333
Copy link

@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
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