-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Conversation
…e injected in HiveConf
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())); |
There was a problem hiding this comment.
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
?
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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!
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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
…e injected in HiveConf (apache#5261)
…e injected in HiveConf (apache#5261)
…e injected in HiveConf (apache#5261)
…e injected in HiveConf (apache#5261)
为什么还报这个问题啊 |
@FengSq 解决了吗? |
Purpose of this pull request
Check list
New License Guide
release-note
.