[jira] [Commented] (FLINK-7704) Port JobPlanHandler to new REST endpoint

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=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

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=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

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=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

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=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

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=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

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=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

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=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

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=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

[Till Rohrmann (JIRA)]

[ 
https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=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

[Hai Zhou UTC+8 (JIRA)]

[ 
https://issues.apache.org/jira/browse/FLINK-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=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)