Github user aljoscha commented on the pull request:
https://github.com/apache/flink/pull/1768#issuecomment-196300410
Thanks a lot! I merged it.
Could you please close this PR if github does not do so automatically.
---
If your project is set up for it, you can reply to this
Github user fijolekProjects commented on the pull request:
https://github.com/apache/flink/pull/1768#issuecomment-196258570
Hi @aljoscha I can confirm that after merging #1583 this change looks good
---
If your project is set up for it, you can reply to this email and have your
reply
Github user aljoscha commented on the pull request:
https://github.com/apache/flink/pull/1768#issuecomment-196251716
Hi @fijolekProjects,
could you please confirm that everything works now that #1583 is merged. If
yes, then I would like to merge this one as well.
---
If your
Github user fijolekProjects commented on the pull request:
https://github.com/apache/flink/pull/1768#issuecomment-195774252
Thanks, I removed backend changes
---
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 aljoscha commented on the pull request:
https://github.com/apache/flink/pull/1768#issuecomment-193730962
Ok, makes sense.
---
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
Github user StephanEwen commented on the pull request:
https://github.com/apache/flink/pull/1768#issuecomment-193730582
I think that #1583 also addresses parts this, in a more generic fashion.
Reading the config earlier should not be a problem.
I would vote to get first
Github user aljoscha commented on the pull request:
https://github.com/apache/flink/pull/1768#issuecomment-193720952
Thanks for your PR! I'll look into whether loading the ExecutionConfig
earlier breaks anything and then merge it.
---
If your project is set up for it, you can reply
Github user aljoscha commented on a diff in the pull request:
https://github.com/apache/flink/pull/1768#discussion_r55342808
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/executiongraph/ExecutionGraph.java
---
@@ -306,6 +306,12 @@ public ExecutionGraph(
GitHub user fijolekProjects opened a pull request:
https://github.com/apache/flink/pull/1768
[FLINK-3583] User configuration visible in webdashboard when job is running
Before: