-
Notifications
You must be signed in to change notification settings - Fork 28.6k
[SPARK-4211][Build] Fixes hive.version in Maven profile hive-0.13.1 #3072
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
Conversation
instead of the Spark compatible `hive.version=0.13.1a` Note: `hive.version=0.13.1a` is the default version. However, when explicitly specifying the `hive-0.13.1` maven profile, the wrong one would be selected. e.g. mvn -Phive -Phive=0.13.1 See PR #2685
Can one of the admins verify this patch? |
test this please |
Test build #22809 has started for PR 3072 at commit
|
Test build #22809 has finished for PR 3072 at commit
|
Test PASSed. |
@coderfi Would you please file a JIRA ticket for this and then rename the title of this PR to something like "[SPARK-9999][Build] Fixes hive.version in Maven profile hive-0.13.1"? |
This LGTM besides PR title and the missing JIRA ticket. Thanks! |
Hello, I'll file a ticket. Fi
|
hive-0.13.1
maven profile now sets hive.version=0.13.1a
LGTM |
Thanks! Merged to master. |
instead of `hive.version=0.13.1`. e.g. mvn -Phive -Phive=0.13.1 Note: `hive.version=0.13.1a` is the default property value. However, when explicitly specifying the `hive-0.13.1` maven profile, the wrong one would be selected. References: PR #2685, which resolved a package incompatibility issue with Hive-0.13.1 by introducing a special version Hive-0.13.1a Author: fi <coderfi@gmail.com> Closes #3072 from coderfi/master and squashes the following commits: 7ca4b1e [fi] Fixes the `hive-0.13.1` maven profile referencing `hive.version=0.13.1` instead of the Spark compatible `hive.version=0.13.1a` Note: `hive.version=0.13.1a` is the default version. However, when explicitly specifying the `hive-0.13.1` maven profile, the wrong one would be selected. e.g. mvn -Phive -Phive=0.13.1 See PR #2685 (cherry picked from commit df607da) Signed-off-by: Michael Armbrust <michael@databricks.com>
Glad to be of service. :) |
Simple pom.xml update.