[GitHub] flink issue #4356: [FLINK-5486] Fix lacking of synchronization in BucketingS...

2017-10-19 Thread GJL
Github user GJL commented on the issue: https://github.com/apache/flink/pull/4356 Thanks for your contribution. I have a few questions because the submitted code does not match with what the ticket currently says: From https://issues.apache.org/jira/browse/FLINK-5486:

[GitHub] flink issue #4356: [FLINK-5486] Fix lacking of synchronization in BucketingS...

2017-10-18 Thread zhangminglei
Github user zhangminglei commented on the issue: https://github.com/apache/flink/pull/4356 @tedyu I am retesting now..let us wait what will happen. ---

[GitHub] flink issue #4356: [FLINK-5486] Fix lacking of synchronization in BucketingS...

2017-10-10 Thread zhangminglei
Github user zhangminglei commented on the issue: https://github.com/apache/flink/pull/4356 OKay. Thanks , Ted. Will retest again soon enough./ ---

[GitHub] flink issue #4356: [FLINK-5486] Fix lacking of synchronization in BucketingS...

2017-10-10 Thread tedyu
Github user tedyu commented on the issue: https://github.com/apache/flink/pull/4356 retest this please ---

[GitHub] flink issue #4356: [FLINK-5486] Fix lacking of synchronization in BucketingS...

2017-08-19 Thread zhangminglei
Github user zhangminglei commented on the issue: https://github.com/apache/flink/pull/4356 triggering now --- 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] flink issue #4356: [FLINK-5486] Fix lacking of synchronization in BucketingS...

2017-08-17 Thread zhangminglei
Github user zhangminglei commented on the issue: https://github.com/apache/flink/pull/4356 Okay. Ted. I will trigger the QA soon. --- 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

[GitHub] flink issue #4356: [FLINK-5486] Fix lacking of synchronization in BucketingS...

2017-08-17 Thread tedyu
Github user tedyu commented on the issue: https://github.com/apache/flink/pull/4356 Run QA again ? --- 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 so, or if

[GitHub] flink issue #4356: [FLINK-5486] Fix lacking of synchronization in BucketingS...

2017-08-03 Thread zhangminglei
Github user zhangminglei commented on the issue: https://github.com/apache/flink/pull/4356 ``` Tests run: 6, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 91.985 sec <<< FAILURE! - in org.apache.flink.test.checkpointing.ExternalizedCheckpointITCase

[GitHub] flink issue #4356: [FLINK-5486] Fix lacking of synchronization in BucketingS...

2017-07-24 Thread tedyu
Github user tedyu commented on the issue: https://github.com/apache/flink/pull/4356 lgtm --- 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 so, or if the feature