Github user iampeter commented on the issue:
https://github.com/apache/flink/pull/2941
I agree with @sachingoel0101 as for overrriding classes from `vendor.css`
---
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 iampeter commented on the issue:
https://github.com/apache/flink/pull/2467
@mushketyk do make sure that the generated files are up-to-date (if they're
not)
---
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 iampeter commented on the issue:
https://github.com/apache/flink/pull/2457
@greghogan right - missed the generated files. @mushketyk I suppose
`gulp`ing will help
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user iampeter commented on the issue:
https://github.com/apache/flink/pull/2457
@mushketyk from my side 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
enabled
Github user iampeter commented on the issue:
https://github.com/apache/flink/pull/2431
@mushketyk I suppose so, right @rmetzger ?
---
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 iampeter commented on a diff in the pull request:
https://github.com/apache/flink/pull/2457#discussion_r87659296
--- Diff:
flink-runtime-web/web-dashboard/app/scripts/modules/jobs/jobs.dir.coffee ---
@@ -281,7 +281,8 @@ angular.module('fli
Github user iampeter commented on a diff in the pull request:
https://github.com/apache/flink/pull/2467#discussion_r87658530
--- Diff:
flink-runtime-web/web-dashboard/app/scripts/modules/jobs/jobs.dir.coffee ---
@@ -168,14 +168,15 @@ angular.module('flinkApp')
Github user iampeter commented on the issue:
https://github.com/apache/flink/pull/2431
@mushketyk looks good, 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
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 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 user iampeter commented on the issue:
https://github.com/apache/flink/pull/2467
Thanks, that's better @mushketyk, but I think would be even better if you
did not identify the elements explicitly by ids ('#canvas', '#job-panel'), but
have the directive
Github user iampeter commented on the issue:
https://github.com/apache/flink/pull/2467
@mushketyk thanks for this feature, very useful indeed.
I'd like to ask you to reconsider some things:
1. First of all, I think it would be good to wrap split.js in an Angular
dire
Github user iampeter commented on the pull request:
https://github.com/apache/flink/pull/1421#issuecomment-162567139
Maybe a top right action button, triggering a Bootstrap dropdown with the
Cancel action?
![action](https://cloud.githubusercontent.com/assets/713801/11631787
GitHub user iampeter opened a pull request:
https://github.com/apache/flink/pull/1346
Fixed FLINK-2942
This should fix https://issues.apache.org/jira/browse/FLINK-2942
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/iampeter
Github user iampeter commented on the pull request:
https://github.com/apache/flink/pull/1236#issuecomment-146333800
@sachingoel0101 @aljoscha looks great, but I would make the following
changes if possible:
1. Move the 'livechart' directive to a sep
Github user iampeter commented on the pull request:
https://github.com/apache/flink/pull/1236#issuecomment-146320512
@aljoscha ok will do
---
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 iampeter commented on the pull request:
https://github.com/apache/flink/pull/1219#issuecomment-145842071
@sachingoel0101 unfortunately haven't had time yet, so I guess you can take
on it
---
If your project is set up for it, you can reply to this email and have
GitHub user iampeter opened a pull request:
https://github.com/apache/flink/pull/1164
[web-dashboard] Added task managers to the new web dashboard
I transfered the task manager REST API code to a RequestHandler
(TaskManagersHandler) and added the JS view.
There are no
GitHub user iampeter opened a pull request:
https://github.com/apache/flink/pull/1148
[FLINK-2357] [web dashboard] Fixed broken plan on second entry
Fixed the issue with wrong rendering when entering the plan for the second
time.
You can merge this pull request into a Git
19 matches
Mail list logo