[GitHub] flink issue #4978: [FLINK-7704][hotfix][flip6] Fix JobPlanInfoTest package p...

2017-11-09 Thread tillrohrmann 
Github user tillrohrmann commented on the issue:

https://github.com/apache/flink/pull/4978
  
LGTM. Thanks for your contribution @yew1eb. Merging.


---

[GitHub] flink issue #4978: [FLINK-7704][hotfix][flip6] Fix JobPlanInfoTest package p...

2017-11-08 Thread yew1eb 
Github user yew1eb commented on the issue:

https://github.com/apache/flink/pull/4978
  
thx @tillrohrmann , removed the redundant import.


---

[GitHub] flink issue #4978: [FLINK-7704][hotfix][flip6] Fix JobPlanInfoTest package p...

2017-11-08 Thread tillrohrmann 
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.
```


---

[GitHub] flink issue #4978: [FLINK-7704][hotfix][flip6] Fix JobPlanInfoTest package p...

2017-11-07 Thread tzulitai 
Github user tzulitai commented on the issue:

https://github.com/apache/flink/pull/4978
  
LGTM 👍 


---