[jira] [Commented] (FLINK-7704) Port JobPlanHandler to new REST endpoint
[ https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16245423#comment-16245423 ] ASF GitHub Bot commented on FLINK-7704: --- Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/4978 > Port JobPlanHandler to new REST endpoint > > > Key: FLINK-7704 > URL: https://issues.apache.org/jira/browse/FLINK-7704 > Project: Flink > Issue Type: Sub-task > Components: Distributed Coordination, REST, Webfrontend >Reporter: Tzu-Li (Gordon) Tai >Assignee: Hai Zhou UTC+8 > Labels: flip-6 > Fix For: 1.4.0 > > > Port existing {{JobPlanHandler}} to new REST endpoint. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (FLINK-7704) Port JobPlanHandler to new REST endpoint
[ https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16245421#comment-16245421 ] ASF GitHub Bot commented on FLINK-7704: --- Github user tillrohrmann commented on the issue: https://github.com/apache/flink/pull/4978 LGTM. Thanks for your contribution @yew1eb. Merging. > Port JobPlanHandler to new REST endpoint > > > Key: FLINK-7704 > URL: https://issues.apache.org/jira/browse/FLINK-7704 > Project: Flink > Issue Type: Sub-task > Components: Distributed Coordination, REST, Webfrontend >Reporter: Tzu-Li (Gordon) Tai >Assignee: Hai Zhou UTC+8 > Labels: flip-6 > Fix For: 1.4.0 > > > Port existing {{JobPlanHandler}} to new REST endpoint. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (FLINK-7704) Port JobPlanHandler to new REST endpoint
[ https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16244342#comment-16244342 ] ASF GitHub Bot commented on FLINK-7704: --- Github user yew1eb commented on the issue: https://github.com/apache/flink/pull/4978 thx @tillrohrmann , removed the redundant import. > Port JobPlanHandler to new REST endpoint > > > Key: FLINK-7704 > URL: https://issues.apache.org/jira/browse/FLINK-7704 > Project: Flink > Issue Type: Sub-task > Components: Distributed Coordination, REST, Webfrontend >Reporter: Tzu-Li (Gordon) Tai >Assignee: Hai Zhou UTC+8 > Labels: flip-6 > Fix For: 1.4.0 > > > Port existing {{JobPlanHandler}} to new REST endpoint. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (FLINK-7704) Port JobPlanHandler to new REST endpoint
[ https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16243682#comment-16243682 ] ASF GitHub Bot commented on FLINK-7704: --- Github user tillrohrmann commented on the issue: https://github.com/apache/flink/pull/4978 There are two checkstyle violations in `JobPlanInfoTest`. ``` [ERROR] src/test/java/org/apache/flink/runtime/rest/messages/JobPlanInfoTest.java:[22,1] (imports) RedundantImport: Redundant import from the same package - org.apache.flink.runtime.rest.messages.JobPlanInfo. [ERROR] src/test/java/org/apache/flink/runtime/rest/messages/JobPlanInfoTest.java:[23,1] (imports) RedundantImport: Redundant import from the same package - org.apache.flink.runtime.rest.messages.RestResponseMarshallingTestBase. ``` > Port JobPlanHandler to new REST endpoint > > > Key: FLINK-7704 > URL: https://issues.apache.org/jira/browse/FLINK-7704 > Project: Flink > Issue Type: Sub-task > Components: Distributed Coordination, REST, Webfrontend >Reporter: Tzu-Li (Gordon) Tai >Assignee: Hai Zhou UTC+8 > Labels: flip-6 > Fix For: 1.4.0 > > > Port existing {{JobPlanHandler}} to new REST endpoint. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (FLINK-7704) Port JobPlanHandler to new REST endpoint
[ https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16243348#comment-16243348 ] ASF GitHub Bot commented on FLINK-7704: --- GitHub user yew1eb opened a pull request: https://github.com/apache/flink/pull/4978 [FLINK-7704][hotfix][flip6] Fix JobPlanInfoTest package path This PR fix the package path of `JobPlanInfoTest`, consistent with the JobPlanInfo `org.apache.flink.runtime.rest.messages`. You can merge this pull request into a Git repository by running: $ git pull https://github.com/yew1eb/flink FLINK-7704-hotfix Alternatively you can review and apply these changes as the patch at: https://github.com/apache/flink/pull/4978.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 #4978 commit fecaeeb14156867021d678f3f7e64776839f545a Author: yew1eb Date: 2017-11-08T03:49:05Z [FLINK-7704][hotfix][flip6] Fix JobPlanInfoTest package path > Port JobPlanHandler to new REST endpoint > > > Key: FLINK-7704 > URL: https://issues.apache.org/jira/browse/FLINK-7704 > Project: Flink > Issue Type: Sub-task > Components: Distributed Coordination, REST, Webfrontend >Reporter: Tzu-Li (Gordon) Tai >Assignee: Hai Zhou UTC+8 > Labels: flip-6 > Fix For: 1.4.0 > > > Port existing {{JobPlanHandler}} to new REST endpoint. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (FLINK-7704) Port JobPlanHandler to new REST endpoint
[ https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16243422#comment-16243422 ] ASF GitHub Bot commented on FLINK-7704: --- Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/4978 LGTM 👍 > Port JobPlanHandler to new REST endpoint > > > Key: FLINK-7704 > URL: https://issues.apache.org/jira/browse/FLINK-7704 > Project: Flink > Issue Type: Sub-task > Components: Distributed Coordination, REST, Webfrontend >Reporter: Tzu-Li (Gordon) Tai >Assignee: Hai Zhou UTC+8 > Labels: flip-6 > Fix For: 1.4.0 > > > Port existing {{JobPlanHandler}} to new REST endpoint. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (FLINK-7704) Port JobPlanHandler to new REST endpoint
[ https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16199343#comment-16199343 ] ASF GitHub Bot commented on FLINK-7704: --- Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/4768 > Port JobPlanHandler to new REST endpoint > > > Key: FLINK-7704 > URL: https://issues.apache.org/jira/browse/FLINK-7704 > Project: Flink > Issue Type: Sub-task > Components: Distributed Coordination, REST, Webfrontend >Reporter: Tzu-Li (Gordon) Tai >Assignee: Hai Zhou UTC+8 > Labels: flip-6 > Fix For: 1.4.0 > > > Port existing {{JobPlanHandler}} to new REST endpoint. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (FLINK-7704) Port JobPlanHandler to new REST endpoint
[ https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16198732#comment-16198732 ] ASF GitHub Bot commented on FLINK-7704: --- Github user yew1eb commented on the issue: https://github.com/apache/flink/pull/4768 CC @tillrohrmann > Port JobPlanHandler to new REST endpoint > > > Key: FLINK-7704 > URL: https://issues.apache.org/jira/browse/FLINK-7704 > Project: Flink > Issue Type: Sub-task > Components: Distributed Coordination, REST, Webfrontend >Reporter: Tzu-Li (Gordon) Tai >Assignee: Hai Zhou UTC+8 > Labels: flip-6 > Fix For: 1.4.0 > > > Port existing {{JobPlanHandler}} to new REST endpoint. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (FLINK-7704) Port JobPlanHandler to new REST endpoint
[ https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16185493#comment-16185493 ] Till Rohrmann commented on FLINK-7704: -- Hi [~haizhou], great to hear that you want to help with Flip-6 :-) All help is highly welcome! Go ahead and do the port of the {{JobPlanHandler}}. > Port JobPlanHandler to new REST endpoint > > > Key: FLINK-7704 > URL: https://issues.apache.org/jira/browse/FLINK-7704 > Project: Flink > Issue Type: Sub-task > Components: Distributed Coordination, REST, Webfrontend >Reporter: Tzu-Li (Gordon) Tai >Assignee: Hai Zhou UTC+8 > Labels: flip-6 > Fix For: 1.4.0 > > > Port existing {{JobPlanHandler}} to new REST endpoint. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (FLINK-7704) Port JobPlanHandler to new REST endpoint
[ https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16184220#comment-16184220 ] Hai Zhou UTC+8 commented on FLINK-7704: --- Hey [~till.rohrmann], I recently learned about flip-6, this new deployment on yarn architecture is very cool! I'd like to try this and contribute flip-6. I review your PR(_FLINK-7695 Port JobConfigHandler to new REST endpoint_) and I want fix this ticket after it. Do you allow me to try it? Thanks! > Port JobPlanHandler to new REST endpoint > > > Key: FLINK-7704 > URL: https://issues.apache.org/jira/browse/FLINK-7704 > Project: Flink > Issue Type: Sub-task > Components: Distributed Coordination, REST, Webfrontend >Reporter: Tzu-Li (Gordon) Tai >Assignee: Hai Zhou UTC+8 > Labels: flip-6 > Fix For: 1.4.0 > > > Port existing {{JobPlanHandler}} to new REST endpoint. -- This message was sent by Atlassian JIRA (v6.4.14#64029)