Github user mushketyk commented on the issue:
https://github.com/apache/flink/pull/2323
Awesome! Thank you.
---
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
Github user StephanEwen commented on the issue:
https://github.com/apache/flink/pull/2323
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
Github user mushketyk commented on the issue:
https://github.com/apache/flink/pull/2323
@StephanEwen I've updated the PR according to your review.
---
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
Github user mushketyk commented on the issue:
https://github.com/apache/flink/pull/2323
@StephanEwen Sorry, somehow I missed your comment. I'll update the PR today.
---
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
Github user StephanEwen commented on the issue:
https://github.com/apache/flink/pull/2323
@mushketyk Are you going to update this pull request?
---
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
Github user StephanEwen commented on the issue:
https://github.com/apache/flink/pull/2323
Looks good. Can you remove the Guava dependency, though? We try to avoid
Guava as much as possible, because it causes too many dependency issues...
---
If your project is set up for it, you can
Github user mushketyk commented on the issue:
https://github.com/apache/flink/pull/2323
Set maximum limit for the toString result, as suggested by Stephan here:
https://issues.apache.org/jira/browse/FLINK-2090
---
If your project is set up for it, you can reply to this email and