[GitHub] flink issue #3274: [FLINK-5723][UI]Use Used instead of Initial to make taskm...
Github user WangTaoTheTonic commented on the issue: https://github.com/apache/flink/pull/3274 sure. Fixed in - 1.2.1 via e3e3c2a7f9c8dd8576e0e27b2efddb7ff42c7c0d - 1.3.0 via 03e6c249156fbbfeef39397a70c70bb905469d09 --- 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 issue #3274: [FLINK-5723][UI]Use Used instead of Initial to make taskm...
Github user uce commented on the issue: https://github.com/apache/flink/pull/3274 @WangTaoTheTonic We forgot to close this PR while merging. Could you please close this manually? --- 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 issue #3274: [FLINK-5723][UI]Use Used instead of Initial to make taskm...
Github user StephanEwen commented on the issue: https://github.com/apache/flink/pull/3274 Looks good, merging this... --- 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 issue #3274: [FLINK-5723][UI]Use Used instead of Initial to make taskm...
Github user WangTaoTheTonic commented on the issue: https://github.com/apache/flink/pull/3274 @StephanEwen What do you think about this? Could you help merge this if you're ok with it? 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 issue #3274: [FLINK-5723][UI]Use Used instead of Initial to make taskm...
Github user zentol commented on the issue: https://github.com/apache/flink/pull/3274 +1 from my side. --- 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 issue #3274: [FLINK-5723][UI]Use Used instead of Initial to make taskm...
Github user WangTaoTheTonic commented on the issue: https://github.com/apache/flink/pull/3274 @zentol Is it ok to go? --- 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 issue #3274: [FLINK-5723][UI]Use Used instead of Initial to make taskm...
Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/3274 AFAIK, for simple changes we don't strictly require a separate PR for backporting to a release branch. Committers usually just cherry-pick the changes on the release branch. --- 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 issue #3274: [FLINK-5723][UI]Use Used instead of Initial to make taskm...
Github user WangTaoTheTonic commented on the issue: https://github.com/apache/flink/pull/3274 Thanks. That 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 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 issue #3274: [FLINK-5723][UI]Use Used instead of Initial to make taskm...
Github user greghogan commented on the issue: https://github.com/apache/flink/pull/3274 I don't know the best workflow for multi-release PRs. It can work either way, so hopefully another developer will chime in here. I think we tend to only create a single PR for simple changes and the committer can patch the changes onto the stable branches. --- 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 issue #3274: [FLINK-5723][UI]Use Used instead of Initial to make taskm...
Github user WangTaoTheTonic commented on the issue: https://github.com/apache/flink/pull/3274 Thanks @greghogan. Should I make another PR against branch `release-1.2`? --- 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 issue #3274: [FLINK-5723][UI]Use Used instead of Initial to make taskm...
Github user greghogan commented on the issue: https://github.com/apache/flink/pull/3274 Travis is effectively passing. The tests are duplicated under multiple environments. This would also be a nice fix for 1.2.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 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 issue #3274: [FLINK-5723][UI]Use Used instead of Initial to make taskm...
Github user WangTaoTheTonic commented on the issue: https://github.com/apache/flink/pull/3274 Travis timeout :) How to kick it up again? --- 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 issue #3274: [FLINK-5723][UI]Use Used instead of Initial to make taskm...
Github user WangTaoTheTonic commented on the issue: https://github.com/apache/flink/pull/3274 @zentol Thanks for review. I've changed .jade file, but it looks like CI is not up properly :( --- 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 issue #3274: [FLINK-5723][UI]Use Used instead of Initial to make taskm...
Github user zentol commented on the issue: https://github.com/apache/flink/pull/3274 you have to modify the corresponding jade files; modifications to the html files will be overridden the next time someone builds the web-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. ---