Github user tzulitai commented on the issue:
https://github.com/apache/flink/pull/4691
Thanks a lot for the review @tillrohrmann. I'll go ahead and merge this PR
after addressing your comments.
---
Github user tzulitai commented on the issue:
https://github.com/apache/flink/pull/4691
Sorry about that, local Travis tests pass, should be fine now.
---
Github user tillrohrmann commented on the issue:
https://github.com/apache/flink/pull/4691
Still failing with `ClusterConfigHandlerTest.testGetPaths:32 NullPointer`
---
Github user tzulitai commented on the issue:
https://github.com/apache/flink/pull/4691
@tillrohrmann I've corrected checkstyles violations and rebased.
---
Github user tillrohrmann commented on the issue:
https://github.com/apache/flink/pull/4691
Thanks for your contribution @tzulitai. There are still some checkstyle
violations in the code.
---