-
Notifications
You must be signed in to change notification settings - Fork 28.3k
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
SPARK-4532: Fix bug in detection of Hive in Spark 1.2 #3398
Conversation
Because the Hive profile is no longer defined in the root pom, we need to check specifically in the sql/hive pom when we perform the check in make-distribtion.sh.
Test build #23710 has started for PR 3398 at commit
|
LGTM |
Thanks @liancheng |
@pwendell, I use command as follows: or mvn help:evaluate -Dexpression=project.activeProfiles -pl sql/hive -Phadoop-2.3 -Phive-0.13.1 -Phive-thriftserver -Pyarn -Dyarn.version=2.3.0-cdh5.1.2 -Dhadoop.version=2.3.0-cdh5.1.2 2>/dev/null | grep -v "INFO" | fgrep --count "hive-0.13.1" it still return 0, if use command as follows: |
@zzcclp You also need to add |
@liancheng, thanks |
Test build #23710 has finished for PR 3398 at commit
|
Test PASSed. |
Because the Hive profile is no longer defined in the root pom, we need to check specifically in the sql/hive pom when we perform the check in make-distribtion.sh. Author: Patrick Wendell <pwendell@gmail.com> Closes #3398 from pwendell/make-distribution and squashes the following commits: 8a58279 [Patrick Wendell] Fix bug in detection of Hive in Spark 1.2 (cherry picked from commit a81918c) Signed-off-by: Patrick Wendell <pwendell@gmail.com>
Because the Hive profile is no longer defined in the root pom,
we need to check specifically in the sql/hive pom when we
perform the check in make-distribtion.sh.