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:
[email protected]


With regards,
Apache Git Services

Reply via email to