Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/895#discussion_r34262145
--- Diff: flink-tests/src/test/resources/log4j-test.properties ---
@@ -18,7 +18,7 @@
# Set root logger level to OFF to not flood build logs
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/897#issuecomment-120279506
+1 to merge
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
GitHub user rmetzger opened a pull request:
https://github.com/apache/flink/pull/895
[FLINK-2008][FLINK-2296] Fix checkpoint committing KafkaITCase [wip]
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/rmetzger/flink flink2008
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/903#issuecomment-120702067
+1 looks good
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/904#issuecomment-120702048
Are there any other possible options for the options field except for the
parallelism?
I think users might be confused how the options and arguments differ
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1366#issuecomment-157504656
thank you for the contribution, the change looks good.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1366#issuecomment-157506571
Did you test the change?
I've tried it but got the following error
```
Removing intermediate container 0b4ba3a6115e
Step 3 : RUN mkdir
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/750#issuecomment-157833952
There is a readme:
https://github.com/apache/flink/tree/master/flink-runtime-web
---
If your project is set up for it, you can reply to this email and have your
reply
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/1342#discussion_r44767927
--- Diff: flink-dist/src/main/flink-bin/bin/config.sh ---
@@ -249,7 +249,15 @@ if [ -n "$HADOOP_HOME" ]; then
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1342#issuecomment-156388624
Thank you for addressing my concerns so quickly
+1 to merge.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1338#issuecomment-156450181
here is the jar file:
https://transfer.sh/KNAnm/flink0.10-scala2.11-1.0-snapshot.jar
it has been build from here:
https://github.com/rmetzger/scratch/tree
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1338#issuecomment-156432901
I think we need to improve a bit on the error messages.
This is what I got for submitting a job without arguments on the web
interface:
```
Error
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1338#issuecomment-156433498
Even with all the fields filled, it was not starting:
http://i.imgur.com/5WXKw9N.png
the log file also contained no helpful information
---
If your project
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1342#issuecomment-156423600
Amazing, thank you.
I'm going to merge you change as soon as Travis gives me green light ;)
I've squashed and rebased your commit and will push it probably
GitHub user rmetzger opened a pull request:
https://github.com/apache/flink/pull/1062
[FLINK-2326] Write yarn properties file to temp directory
Flink's YARN client is writing a `.yarn-properties` file to the `conf/`
directory containing the JobManager's host and port.
It seems
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1070#issuecomment-135501709
Looks good, +1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1048#issuecomment-135770247
That sounds very good.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1048#issuecomment-135812417
Sending a stringified exception is not an option between jm and client
because the user might rely on the exception in the RemoteExecEnv.
---
If your project is set
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1075#issuecomment-136129581
Thanks a lot for the contribution.
Can you add a test case for the method to make sure the issue is not
re-introduced again when somebody else is changing the code
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/commit/554b77bcd9ed66d57d99d4990774a43f35f6a835#commitcomment-12965859
Currently, flink-runtime has a dependency on Hadoop, so I can assume its
always available.
Even for a binary Flink release
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1058#issuecomment-134980282
Are you sure this is a fix for FLINK-2512 ?
I don't see how this change is fixing any issue.
---
If your project is set up for it, you can reply to this email
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1038#issuecomment-134584589
I agree. Lets file a JIRA and do it separately, as this is probably a
bigger task.
---
If your project is set up for it, you can reply to this email and have your
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1038#issuecomment-134531698
@mxm: I removed the comment.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1038#issuecomment-135013779
Thanks alot!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/1064#discussion_r38078423
--- Diff: flink-staging/flink-hcatalog/pom.xml ---
@@ -34,17 +34,64 @@ under the License.
packagingjar/packaging
+ repositories
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/1064#discussion_r38078384
--- Diff: flink-staging/flink-hcatalog/pom.xml ---
@@ -34,17 +34,64 @@ under the License.
packagingjar/packaging
+ repositories
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1062#issuecomment-135372331
The travis build passed btw.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1062#issuecomment-135371903
Can you check the logs of `application_1440646275804_0005`? They are
located in `target/yarn-...`.
---
If your project is set up for it, you can reply to this email
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1062#issuecomment-135363533
Lets first wait for the builds to pass.
The errors are actually not unrelated to my change.
---
If your project is set up for it, you can reply to this email
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1048#issuecomment-135363969
I addressed all PR comments.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1038#issuecomment-135343326
I'm manually closing this pull request. It has been merged by @uce.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user rmetzger closed the pull request at:
https://github.com/apache/flink/pull/1038
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user rmetzger closed the pull request at:
https://github.com/apache/flink/pull/1055
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1055#issuecomment-135327605
Thank you for merging it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1076#issuecomment-136339437
Make sure it does not break with @tillrohrmann's recent merge ;)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
GitHub user rmetzger opened a pull request:
https://github.com/apache/flink/pull/1082
[FLINK-2372] Add new FlinkKafkaProducer
This pull request is reworking Flink's KafkaSink.
I've renamed the class to `FlinkKafkaProducer` to match it with
`FlinkKafkaConsumer`.
It uses
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/948#issuecomment-136377793
Thank you for the response. I was asking because I think its a requirement
that the JobManager is running as a mesos task in the cluster as well.
But as far as I
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1082#issuecomment-13154
Thank you for the feedback. I will address the concerns.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1082#issuecomment-136669637
I updated the PR and rebased to master (which is adding some commits from
the future (at least from GitHubs perspective) )
---
If your project is set up for it, you
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/750#discussion_r38511039
--- Diff:
flink-runtime/src/main/scala/org/apache/flink/runtime/taskmanager/TaskManager.scala
---
@@ -411,6 +411,23 @@ class TaskManager
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/750#discussion_r38511106
--- Diff: docs/apis/cli.md ---
@@ -185,6 +189,18 @@ Action "list" lists running and scheduled programs.
-s,--scheduled
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/1046#discussion_r38844802
--- Diff:
flink-contrib/flink-storm-compatibility/flink-storm-compatibility-examples/src/main/java/org/apache/flink/stormcompatibility/excamation
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/1046#discussion_r38844654
--- Diff:
flink-contrib/flink-storm-compatibility/flink-storm-compatibility-core/src/test/java/org/apache/flink/stormcompatibility/wrappers
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/1046#discussion_r38844529
--- Diff:
flink-contrib/flink-storm-compatibility/flink-storm-compatibility-core/src/main/java/org/apache/flink/stormcompatibility/wrappers
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/750#discussion_r38856545
--- Diff: docs/apis/cli.md ---
@@ -185,6 +189,18 @@ Action "list" lists running and scheduled programs.
-s,--scheduled
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/750#discussion_r38863585
--- Diff: docs/apis/cli.md ---
@@ -185,6 +189,18 @@ Action "list" lists running and scheduled programs.
-s,--scheduled
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1100#issuecomment-138313755
Thank you for opening a new pull request.
+1 to merge (@ merging committer: please rename the commit to include the
JIRA id ;) )
---
If your project is set
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1107#issuecomment-138600722
Thanks a lot for opening the pull request.
Can you change the behavior of this a little bit to make the `-n` argument
a required argument again? (if the config
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1084#issuecomment-137390414
I'm currently trying out the module. Some comments:
- Why do we name the module `flink-connector-hdfs`. I think a name such as
`flink-connector-filesystems
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/1084#discussion_r38627730
--- Diff: docs/apis/streaming_guide.md ---
@@ -1836,6 +1837,110 @@ More about information about Elasticsearch can be
found [here](https://elastic.c
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/1084#discussion_r38435351
--- Diff:
flink-staging/flink-streaming/flink-streaming-connectors/flink-connector-hdfs/pom.xml
---
@@ -0,0 +1,107 @@
+
+
+http
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1085#issuecomment-137095491
+1 to merge
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1091#issuecomment-137679740
Thank you for the contribution. We'll soon review your pull request.
---
If your project is set up for it, you can reply to this email and have your
reply appear
GitHub user rmetzger opened a pull request:
https://github.com/apache/flink/pull/1055
[FLINK-2386] Add new Kafka Consumer for Flink 0.9.1
This is PR https://github.com/apache/flink/pull/1039 against the
**release-0.9** branch!
You can merge this pull request into a Git repository
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1039#issuecomment-134584442
I addressed @hsaputra's comments rebased to the latest master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/948#discussion_r38957167
--- Diff:
flink-mesos/src/main/scala/org/apache/flink/mesos/scheduler/SchedulerUtils.scala
---
@@ -0,0 +1,358 @@
+/*
+ * Licensed to the Apache
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/948#issuecomment-138640533
I'm trying out the code again in GCE ;)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/858#discussion_r38951273
--- Diff:
flink-tests/src/test/scala/org/apache/flink/api/scala/runtime/jobmanager/JobManagerFailsITCase.scala
---
@@ -53,41 +54,41 @@ class
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/948#discussion_r38958292
--- Diff:
flink-mesos/src/main/scala/org/apache/flink/mesos/scheduler/SchedulerUtils.scala
---
@@ -0,0 +1,358 @@
+/*
+ * Licensed to the Apache
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/948#issuecomment-138649089
it seems that the taskmanagers are failing and the scheduler keeps
scheduling new ones: http://i.imgur.com/iZiYa4u.png
---
If your project is set up for it, you
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/941#issuecomment-139186364
I will rebase and merge it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
GitHub user rmetzger opened a pull request:
https://github.com/apache/flink/pull/1113
[FLINK-2639] Add repository for hdp specific jetty to 'vendor-repos' profile
This adds a second hortonworks repository to the `vendor-repos` profile.
Certain HDP-specific Hadoop releases depend
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/750#issuecomment-139194317
> The cancel only sets an internal variable.
That is probably true for all sources that Flink provides.
But the interface for sources is a public
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1101#issuecomment-139178919
+1 to merge.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1078#issuecomment-138825531
Thanks a lot for the pull request.
Sorry that nobody looked at it yet. It seems that all committers are
currently very busy. I'm sure somebody will give you soon
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1078#issuecomment-138826497
Your build is failing due to scalastyle checks
```
[INFO]
[INFO] --- maven-failsafe-plugin:2.17:verify (default) @ flink-ml ---
[INFO] Failsafe
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1108#issuecomment-138820891
+1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1114#issuecomment-139223104
+1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1107#issuecomment-138885088
I agree with your proposal! If `yarn.defaultNumberOfTaskManagers` is set to
an invalid value, we should make `-n` required.
---
If your project is set up for it, you
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/1116#discussion_r39173067
--- Diff:
flink-java/src/main/java/org/apache/flink/api/java/utils/ParameterTool.java ---
@@ -136,15 +157,35 @@ public static ParameterTool
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1116#issuecomment-139280931
Thanks a lot for adding the Javadocs to the class. Good work!
I have some comments in the diff.
---
If your project is set up for it, you can reply to this email
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/1116#discussion_r39173366
--- Diff:
flink-java/src/main/java/org/apache/flink/api/java/utils/ParameterTool.java ---
@@ -234,6 +295,10 @@ public long getLong(String key
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/1116#discussion_r39173008
--- Diff:
flink-java/src/main/java/org/apache/flink/api/java/utils/ParameterTool.java ---
@@ -124,6 +138,13 @@ else if(arg.startsWith
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/1116#discussion_r39173169
--- Diff:
flink-java/src/main/java/org/apache/flink/api/java/utils/ParameterTool.java ---
@@ -124,6 +138,13 @@ else if(arg.startsWith
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/1116#discussion_r39172874
--- Diff:
flink-java/src/main/java/org/apache/flink/api/java/utils/ParameterTool.java ---
@@ -124,6 +138,13 @@ else if(arg.startsWith
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/#issuecomment-138928137
+1 Looks good to merge
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
GitHub user rmetzger opened a pull request:
https://github.com/apache/flink/pull/1117
[FLINK-2656] Fix behavior of FlinkKafkaConsumer for out of range offsets
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/rmetzger/flink
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1128#issuecomment-140153585
+1 to merge
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1134#issuecomment-140663091
Big change, but I didn't spot anything suspicious.
+1 to merge
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1135#issuecomment-140730955
There is a test failure
```
StormWrapperSetupHelperTest.testCreateTopologyContext:190 û ClassCast
class su...
```
---
If your project is set up
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1084#issuecomment-140732373
I think the pull request has grown quite a lot. I think we should merge it
now and then improve it from there.
---
If your project is set up for it, you can reply
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1132#issuecomment-140748694
+1 to merge
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/1084#discussion_r39624668
--- Diff: .travis.yml ---
@@ -19,9 +19,9 @@ matrix:
- jdk: "oraclejdk7" # this will also deploy a uberjar to s3 at some
point
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/885#issuecomment-140731019
I agree. We need to use the script.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/1135#discussion_r39624182
--- Diff:
flink-contrib/flink-storm-compatibility/flink-storm-compatibility-core/src/main/java/org/apache/flink/stormcompatibility/api/FlinkTopology.java
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/1084#discussion_r39640862
--- Diff: .travis.yml ---
@@ -19,9 +19,9 @@ matrix:
- jdk: "oraclejdk7" # this will also deploy a uberjar to s3 at some
point
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/1117#discussion_r39251890
--- Diff:
flink-staging/flink-streaming/flink-streaming-connectors/flink-connector-kafka/src/main/java/org/apache/flink/streaming/connectors/kafka/internals
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1117#issuecomment-139491178
Sorry Henry. I've updated the description.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1117#issuecomment-139529893
Thank you for the review. I'll merge it
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1121#issuecomment-141016403
Sorry for the late response, I didn't see this new pull request.
The failed test is okay. The test is known for being unstable.
---
If your project is set up
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1121#issuecomment-141019208
For others who follow this PR, there was already some discussion on this
change here: https://github.com/apache/flink/pull/1107
---
If your project is set up
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1139#issuecomment-141035558
I agree with this approach, so +1 to merge ;)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1139#issuecomment-141050563
Another follow-up todo is adding support for yarn:
```
11:31:01,955 ERROR org.apache.flink.runtime.jobmanager.JobManager
- WebServer could
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/1121#discussion_r39725184
--- Diff: flink-dist/src/main/resources/flink-conf.yaml ---
@@ -130,6 +130,15 @@ state.backend: jobmanager
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1139#issuecomment-141018736
Very nice.
I tried the new interface locally, and it seems to work.
I suspect these values are still test values? http://i.imgur.com/d9ZRR7g.png
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/1121#discussion_r39725238
--- Diff:
flink-yarn-tests/src/main/java/org/apache/flink/yarn/YARNSessionFIFOITCase.java
---
@@ -111,6 +111,20 @@ public void testClientStartup
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1136#issuecomment-141168778
Hi @srowen, thank you for notifying us. We were trying out
https://github.com/databricks/spark-pr-dashboard on Google App Engine (because
we are also getting more
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1134#issuecomment-141167625
Hey @rxin, I'm really sorry that this happened. Max told me that he stopped
the application on AppEngine as soon as he learned that the tool is not working
as expected
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1125#issuecomment-140029908
The pull request should also update the documentation. In particular these
configuration values affect the documentation in many places.
---
If your project is set up
Github user rmetzger commented on a diff in the pull request:
https://github.com/apache/flink/pull/593#discussion_r40493024
--- Diff:
flink-tests/src/test/java/org/apache/flink/test/classloading/ClassLoaderITCase.java
---
@@ -51,12 +50,25 @@ public void
701 - 800 of 2231 matches
Mail list logo