[GitHub] flink pull request: [FLINK-2205] Fix confusing entries in JM UI Jo...
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/927#issuecomment-127511223 I think it is good to merge. The patch for the new UI has already been merged... --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] flink pull request: [FLINK-2205] Fix confusing entries in JM UI Jo...
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/927 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] flink pull request: [FLINK-2205] Fix confusing entries in JM UI Jo...
Github user fhueske commented on the pull request: https://github.com/apache/flink/pull/927#issuecomment-127426412 This PR is fixing the older UI and still valid right? Any reason not to 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 not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] flink pull request: [FLINK-2205] Fix confusing entries in JM UI Jo...
Github user ebautistabar commented on the pull request: https://github.com/apache/flink/pull/927#issuecomment-126099697 As the new UI didn't show job configurations yet, I have created another PR to deal with it, which also includes the display changes requested in FLINK-2205. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] flink pull request: [FLINK-2205] Fix confusing entries in JM UI Jo...
Github user ebautistabar commented on the pull request: https://github.com/apache/flink/pull/927#issuecomment-125109774 Great. I'll add the changes to the new frontend too. Thanks. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] flink pull request: [FLINK-2205] Fix confusing entries in JM UI Jo...
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/927#issuecomment-124109471 I think you can actually add the code to the current UI. It is not going to disappear in the near future, it will probably stay as a fallback variant for one more version. You can add this functionality there. Adding it to the new web frontend is a separate effort. If you want, you can be involved there as well. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] flink pull request: [FLINK-2205] Fix confusing entries in JM UI Jo...
Github user tillrohrmann commented on the pull request: https://github.com/apache/flink/pull/927#issuecomment-124019229 @ebautistabar you're the only one who can close the PR. But maybe the new web interface needs the same fixes as you've proposed for the old UI. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] flink pull request: [FLINK-2205] Fix confusing entries in JM UI Jo...
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/927#issuecomment-123700358 The job manager UI is under rework right now, there is a new version coming up, see here: https://github.com/apache/flink/tree/master/flink-runtime-web This fix (if going to the old UI) would be only temporary... --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] flink pull request: [FLINK-2205] Fix confusing entries in JM UI Jo...
GitHub user ebautistabar opened a pull request: https://github.com/apache/flink/pull/927 [FLINK-2205] Fix confusing entries in JM UI Job Config. section Default display for 'Number of execution retries' is now 'deactivated' and for 'Job parallelism' is 'auto', as suggested in JIRA. You can merge this pull request into a Git repository by running: $ git pull https://github.com/ebautistabar/flink change-job-config-display Alternatively you can review and apply these changes as the patch at: https://github.com/apache/flink/pull/927.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #927 commit ba7534ce56d5329a25d484d0e06aa36363781a65 Author: Enrique Bautista ebautista...@gmail.com Date: 2015-07-21T16:52:05Z [FLINK-2205] Fix confusing entries in JM UI Job Config. section Default display for 'Number of execution retries' is now 'deactivated' and for 'Job parallelism' is 'auto', as suggested in JIRA. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---