[GitHub] zentol opened a new pull request #7451: [FLINK-11270][build] Do not include hadoop in flink-dist by default

[GitBox <gi...@apache.org>]

zentol opened a new pull request #7451:  [FLINK-11270][build] Do not include hadoop in flink-dist by default
URL: https://github.com/apache/flink/pull/7451
 
 
   Based on #7417. Please ignore commit for FLINK-11268.
   
   ## What is the purpose of the change
   
   With this PR hadoop is not included in flink-dist by default, i.e. the `include-hadoop` profile is no longer activated by default.
   
   ## Brief change log
   
   * update `create_binary_release.sh` to reflect profile activation changes
   * remove hadoop-free WordCount E2E test, since this is now the default
   * update `YarnTestBase` to manually copy hadoop to flink-dist
     * These tests directly use flink-dist, similar to e2e tests. They currently rely on hadoop being bundled by before, so I had to change them to explicitly copy hadoop. This required an additional dependency on `flink-shaded-hadoop2-uber` for `flink-yarn-tests`. The tests have additionally been modified to create a copy of flink-dist to ensure we don't taint the environment they are run in.
   
   ## Verifying this change
   
   The release scripts have to be verified manually.
   
   The Yarn tests check that copying the `shaded-hadoop2-uber` jar is sufficient for running hadoop-dependent jobs.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services