-
Notifications
You must be signed in to change notification settings - Fork 28k
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-41392][BUILD] Make maven build Spark master with Hadoop 3.4.0-SNAPSHOT successful #38974
Conversation
https://github.com/LuciferYang/make-distribution.sh/blob/master/.github/workflows/blank.yml
https://github.com/LuciferYang/make-distribution.sh/actions/runs/3645098350 c1539ee (with scala-maven-plugin 4.8.0)build with above Github Actions success |
Run
locally with
SUCCESS
|
Downgrading scala-maven-plugin will reach 4.7.2, and the local maven build will still pass. I will update pr and test the GA compilation
|
https://github.com/LuciferYang/make-distribution.sh/actions/runs/3645502265/jobs/6155706406 9706cf5 (with scala-maven-plugin 4.7.2) test build with Github Actions successful |
This pr can fix the compile issue in the dev mail list reported by @steveloughran, but should we wait until Hadoop 3.4 is upgraded?. What do you think? @HyukjinKwon @dongjoon-hyun @srowen |
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.
Since we don't want to depend on any SNAPSHOT dependencies, let's wait for the official Apache Hadoop 3.4.0, @LuciferYang .
fine to me, close first ~ |
yeah, not going to happen for a while; 3.3.5 RC0 coming soon though; just trying to wrap up an abfs prefetch bug |
time to revisit this; 3.4.0 is in RC phase |
…`sql/core` module for Hadoop 3.4.0 ### What changes were proposed in this pull request? Adds bouncy-castle jdk18 artifacts to test builds in spark-sql. Based on #38974 * only applies the test import changes * dependencies are those of #44359 ### Why are the changes needed? Forthcoming Hadoop 3.4.0 release doesn't export the bouncy-castle JARs; maven builds fail. ### Does this PR introduce _any_ user-facing change? No: test time dependency declarations only. ### How was this patch tested? This was done through the release build/test project https://github.com/apache/hadoop-release-support 1. Latest RC2 artifacts pulled from apache maven staging 2. Spark maven build triggered with the hadoop-version passed down. 3. The 3.3.6 release template worked with spark master (as it should!) 4. With this change the 3.4.0 RC build worked with this change Note: have not *yet* done a maven test run through this yet ### Was this patch authored or co-authored using generative AI tooling? No Closes #45317 from steveloughran/SPARK-41392-HADOOP-3.4.0. Authored-by: yangjie01 <yangjie01@baidu.com> Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
…`sql/core` module for Hadoop 3.4.0 ### What changes were proposed in this pull request? Adds bouncy-castle jdk18 artifacts to test builds in spark-sql. Based on apache#38974 * only applies the test import changes * dependencies are those of apache#44359 ### Why are the changes needed? Forthcoming Hadoop 3.4.0 release doesn't export the bouncy-castle JARs; maven builds fail. ### Does this PR introduce _any_ user-facing change? No: test time dependency declarations only. ### How was this patch tested? This was done through the release build/test project https://github.com/apache/hadoop-release-support 1. Latest RC2 artifacts pulled from apache maven staging 2. Spark maven build triggered with the hadoop-version passed down. 3. The 3.3.6 release template worked with spark master (as it should!) 4. With this change the 3.4.0 RC build worked with this change Note: have not *yet* done a maven test run through this yet ### Was this patch authored or co-authored using generative AI tooling? No Closes apache#45317 from steveloughran/SPARK-41392-HADOOP-3.4.0. Authored-by: yangjie01 <yangjie01@baidu.com> Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
…`sql/core` module for Hadoop 3.4.0 ### What changes were proposed in this pull request? Adds bouncy-castle jdk18 artifacts to test builds in spark-sql. Based on apache#38974 * only applies the test import changes * dependencies are those of apache#44359 ### Why are the changes needed? Forthcoming Hadoop 3.4.0 release doesn't export the bouncy-castle JARs; maven builds fail. ### Does this PR introduce _any_ user-facing change? No: test time dependency declarations only. ### How was this patch tested? This was done through the release build/test project https://github.com/apache/hadoop-release-support 1. Latest RC2 artifacts pulled from apache maven staging 2. Spark maven build triggered with the hadoop-version passed down. 3. The 3.3.6 release template worked with spark master (as it should!) 4. With this change the 3.4.0 RC build worked with this change Note: have not *yet* done a maven test run through this yet ### Was this patch authored or co-authored using generative AI tooling? No Closes apache#45317 from steveloughran/SPARK-41392-HADOOP-3.4.0. Authored-by: yangjie01 <yangjie01@baidu.com> Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
…`sql/core` module for Hadoop 3.4.0 ### What changes were proposed in this pull request? Adds bouncy-castle jdk18 artifacts to test builds in spark-sql. Based on apache#38974 * only applies the test import changes * dependencies are those of apache#44359 ### Why are the changes needed? Forthcoming Hadoop 3.4.0 release doesn't export the bouncy-castle JARs; maven builds fail. ### Does this PR introduce _any_ user-facing change? No: test time dependency declarations only. ### How was this patch tested? This was done through the release build/test project https://github.com/apache/hadoop-release-support 1. Latest RC2 artifacts pulled from apache maven staging 2. Spark maven build triggered with the hadoop-version passed down. 3. The 3.3.6 release template worked with spark master (as it should!) 4. With this change the 3.4.0 RC build worked with this change Note: have not *yet* done a maven test run through this yet ### Was this patch authored or co-authored using generative AI tooling? No Closes apache#45317 from steveloughran/SPARK-41392-HADOOP-3.4.0. Authored-by: yangjie01 <yangjie01@baidu.com> Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
What changes were proposed in this pull request?
This pr aims to add bc-java related to test dependencies to sql module to make maven build Spark master with Hadoop 3.4.0-SNAPSHOT successful.
Why are the changes needed?
Make maven build Spark master with Hadoop 3.4.0-SNAPSHOT successful.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Before
Failed with
org.bouncycastle.jce.provider.BouncyCastleProvider
ClassNotFoundException.After
BUILD SUCCESS