-
Notifications
You must be signed in to change notification settings - Fork 13k
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
[FLINK-13110][hive] add shim of SimpleGenericUDAFParameterInfo for Hive 1.2.1 and 2.3.4 #9001
Conversation
Thanks a lot for your contribution to the Apache Flink project. I'm the @flinkbot. I help the community Review Progress
Please see the Pull Request Review Guide for a full explanation of the review process. The Bot is tracking the review progress through labels. Labels are applied according to the order of the review items. For consensus, approval by a Flink committer of PMC member is required Bot commandsThe @flinkbot bot supports the following commands:
|
Ping @xuefuz @bowenli86 @zjuwangg for review. Meanwhile I'd suggest we hold on changes to hive connector until the build is fixed. |
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.
LGTM. Thanks for fixing ths.
Probably from now on, we should run hive 1.2.1 profile whenever we introduce a call to a new HIve method before CI covers this part of test.
LGTM, @lirui-apache thanks for fixing the problem! I agree that we should be careful and should test hive-1.2.1 profile for every change since now locally before CI runs for hive-1.2.1. Just a side note that we can probably rename the JIRA and PR to something like "add shim of SimpleGenericUDAFParameterInfo for Hive 1.2.1 and 2.3.4" to be more clear on what it's about. @zentol I looked at the build failures in detail. Please correct me if I'm wrong - I believe they are not related, as this PR only affects flink-connector-hive module. |
Thanks @xuefuz and @bowenli86 for the review. I just rebased the PR.
This PR isn't just about creating SimpleGenericUDAFParameterInfo. It also fixes some issue in |
I don't think the build failure is related since the failure is with blink_planner. |
LGTM. I've also changed the name of the jira from "Fix Hive-1.2.1 build" to "add shim of SimpleGenericUDAFParameterInfo for Hive 1.2.1 and 2.3.4". Thanks for your contribution! Merging |
What is the purpose of the change
This PR adds shim of SimpleGenericUDAFParameterInfo for Hive 1.2.1 and 2.3.4
Brief change log
StandardStructObjectInspector.getStructFieldData
Verifying this change
Built and tested against Hive-1.2.1 locally.
Does this pull request potentially affect one of the following parts:
@Public(Evolving)
: noDocumentation