[GitHub] flink issue #2448: [FLINK-3030][web frontend] Enhance dashboard to show exec...

Github user mushketyk commented on the issue: https://github.com/apache/flink/pull/2448 Hi @uce, I've rebased changes and updated the UI according to your suggestion. Now it looks like:

[GitHub] flink issue #2448: [FLINK-3030][web frontend] Enhance dashboard to show exec...

Github user mushketyk commented on the issue: https://github.com/apache/flink/pull/2448 Hi @uce Thank you for your review. I'll update the UI in the next couple of days. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] flink issue #2448: [FLINK-3030][web frontend] Enhance dashboard to show exec...

Github user uce commented on the issue: https://github.com/apache/flink/pull/2448 It worked well, but I'm a little hesitant to merge it as it is. Running with higher parallelism than 1 I think that the button is pretty invasive: ![screen shot 2017-01-27 at 11 50

[GitHub] flink issue #2448: [FLINK-3030][web frontend] Enhance dashboard to show exec...

Github user mushketyk commented on the issue: https://github.com/apache/flink/pull/2448 @rmetzger @iampeter I've rebased the code and updated it according to your suggestions. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] flink issue #2448: [FLINK-3030][web frontend] Enhance dashboard to show exec...

Github user rmetzger commented on the issue: https://github.com/apache/flink/pull/2448 Okay, I'll run gulp when merging. --- 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] flink issue #2448: [FLINK-3030][web frontend] Enhance dashboard to show exec...

Github user iampeter commented on the issue: https://github.com/apache/flink/pull/2448 @rmetzger will you be merging? if so, `flink-runtime-web/web-dashboard/web/js/index.js` just needs a `gulp` --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] flink issue #2448: [FLINK-3030][web frontend] Enhance dashboard to show exec...

Github user iampeter commented on the issue: https://github.com/apache/flink/pull/2448 @mushketyk could you try `npm view gulp-stylus` ? --- 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] flink issue #2448: [FLINK-3030][web frontend] Enhance dashboard to show exec...

Github user StephanEwen commented on the issue: https://github.com/apache/flink/pull/2448 Good to go from my side. @rmetzger and @iampeter - since you are working on the web UI currently, can you try and merge this? --- If your project is set up for it, you can reply to this

[GitHub] flink issue #2448: [FLINK-3030][web frontend] Enhance dashboard to show exec...

Github user mushketyk commented on the issue: https://github.com/apache/flink/pull/2448 Is there anything else that should be updated in this PR? It has been in review for more than a month. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] flink issue #2448: [FLINK-3030][web frontend] Enhance dashboard to show exec...

Github user mushketyk commented on the issue: https://github.com/apache/flink/pull/2448 @StephanEwen Should I change anything else in this PR? --- 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] flink issue #2448: [FLINK-3030][web frontend] Enhance dashboard to show exec...

Github user mushketyk commented on the issue: https://github.com/apache/flink/pull/2448 @StephanEwen I've renamed the class as you suggested. Regarding the CSS change, I don't know why these changes are added. I have the required (according to README.md) `gulp` version: ```

[GitHub] flink issue #2448: [FLINK-3030][web frontend] Enhance dashboard to show exec...

Github user StephanEwen commented on the issue: https://github.com/apache/flink/pull/2448 Good change, all in all, good work! Few minor comments, otherwise +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