zentol opened a new pull request #8805:  [FLINK-12902][hadoop] Remove 
flink-shaded-yarn-tests 
URL: https://github.com/apache/flink/pull/8805
 
 
   Overall, this PR cleans up some dependency issues in `flink-yarn-tests`.
   
   The dependency situation in this module was just insane; you have a few 
unshaded hadoop dependencies from `flink-yarn-tests`, some shaded hadoop 
dependencies from `flink-shaded-yarn-tests` and _yet another_ set of shaded 
hadoop dependencies from `flink-shaded-hadoop`. All of these were available on 
the test classpath, resulting in what is effectively an incomprehensible 
classpath.
   
   This was highlighted in the shading patterns in `flink-shaded-yarn-tests`, 
which absolutely had to be in sync with `flink-shaded-hadoop` as otherwise you 
ended up with strange `VerifyErrors`.
   
   This PR
   a) removes `flink-shaded-yarn-tests` and moves a few of it's (now unshaded) 
dependencies to `flink-yarn-tests`
   b) isolates `flink-yarn-tests`' test classpath from `flink-shaded-hadoop` 
via exclusions
   
   As such, the test classpath now _only_ contains a few unshaded hadoop 
dependencies required for the test. `flink-shaded-hadoop` is only used for 
starting when submitting the applications to YARN, presumably to mirror what 
would be happening in production.
   
   Additionally, this PR adds missing `test` scopes and removes an unused 
jackson dependency.

----------------------------------------------------------------
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]


With regards,
Apache Git Services

Reply via email to