[GitHub] flink issue #5927: [FLINK-8237] [BucketingSink] Better error message added

2018-05-07 Thread fhueske
Github user fhueske commented on the issue: https://github.com/apache/flink/pull/5927 merging ---

[GitHub] flink issue #5927: [FLINK-8237] [BucketingSink] Better error message added

2018-05-07 Thread fhueske
Github user fhueske commented on the issue: https://github.com/apache/flink/pull/5927 Thanks for the update @pavel-shvetsov-git. +1 to merge ---

[GitHub] flink issue #5927: [FLINK-8237] [BucketingSink] Better error message added

2018-05-07 Thread pavel-shvetsov-git
Github user pavel-shvetsov-git commented on the issue: https://github.com/apache/flink/pull/5927 Message has been improved as suggested. ---

[GitHub] flink issue #5927: [FLINK-8237] [BucketingSink] Better error message added

2018-05-03 Thread fhueske
Github user fhueske commented on the issue: https://github.com/apache/flink/pull/5927 Thanks for the PR @pavel-shvetsov-git! I've left a suggestion to improve the error message. Afterwards the PR should be good to merge. ---