Github user asfgit closed the pull request at:
https://github.com/apache/flink/pull/291
---
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 is enabl
Github user StephanEwen commented on the pull request:
https://github.com/apache/flink/pull/291#issuecomment-70415920
Looks good to be merged, in my opinion!
---
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 StephanEwen commented on the pull request:
https://github.com/apache/flink/pull/291#issuecomment-69358452
Good idea. Two comments inline, but otherwise looks good.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user StephanEwen commented on a diff in the pull request:
https://github.com/apache/flink/pull/291#discussion_r22725429
--- Diff:
flink-clients/src/main/java/org/apache/flink/client/web/JobSubmissionServlet.java
---
@@ -146,10 +146,22 @@ protected void doGet(HttpServletRequ
Github user StephanEwen commented on a diff in the pull request:
https://github.com/apache/flink/pull/291#discussion_r22725368
--- Diff:
flink-clients/src/main/java/org/apache/flink/client/web/JobSubmissionServlet.java
---
@@ -146,10 +146,22 @@ protected void doGet(HttpServletRequ
GitHub user qmlmoon opened a pull request:
https://github.com/apache/flink/pull/291
[FLINK-1229][FLINK-1206] Synchronize webclient arguments with command line
and add default parallelism option
You can merge this pull request into a Git repository by running:
$ git pull https
