HyukjinKwon opened a new pull request #31467:
URL: https://github.com/apache/spark/pull/31467


   ### What changes were proposed in this pull request?
   
   This PR is a followup of https://github.com/apache/spark/pull/30701. It uses 
properties of `hadoop-client-api.artifact`, `hadoop-client-runtime.artifact` 
and `hadoop-client-minicluster.artifact` explicitly to set the dependencies and 
versions.
   
   Otherwise, it is logically incorrect. For example, if you build with Hadoop 
2, this dependency becomes `hadoop-client-api:2.7.4` internally, which does not 
exist in Hadoop 2 
(https://mvnrepository.com/artifact/org.apache.hadoop/hadoop-client-api).
   
   ### Why are the changes needed?
   
   - To fix the logical incorrectness.
   - It fixes a potential issue: this actually caused an issue when 
`generate-sources` plugin is used together with Hadoop 2 by default, which 
attempts to pull 2.7.4 of `hadoop-client-api`, `hadoop-client-runtime` and 
`hadoop-client-minicluster` for whatever reason.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No for users and dev. It's more a cleanup.
   
   ### How was this patch tested?
   
   Manually checked the dependencies are correctly placed.
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to