mwalenia commented on a change in pull request #10386: [BEAM-8972] Add Jenkins
job with Combine test for portable Java
URL: https://github.com/apache/beam/pull/10386#discussion_r394218451
##########
File path: .test-infra/jenkins/Flink.groovy
##########
@@ -17,8 +17,8 @@
*/
class Flink {
- private static final String flinkDownloadUrl =
'https://archive.apache.org/dist/flink/flink-1.10.0/flink-1.10.0-bin-scala_2.11.tgz'
- private static final String hadoopDownloadUrl =
'https://repo.maven.apache.org/maven2/org/apache/flink/flink-shaded-hadoop-2-uber/2.8.3-9.0/flink-shaded-hadoop-2-uber-2.8.3-9.0.jar'
+ private static final String flinkDownloadUrl =
'https://archive.apache.org/dist/flink/flink-1.9.1/flink-1.9.1-bin-scala_2.11.tgz'
Review comment:
The CLI of Yarn changed and no longer accepts one of options we supply.
Kamil took care of it in #11146 - I'll remove this commit and rebase onto
master in a while.
----------------------------------------------------------------
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