[GitHub] flink issue #2228: [FLINK-4170][kinesis-connector] Simplify Kinesis connecte...

2016-07-14 Thread rmetzger
Github user rmetzger commented on the issue: https://github.com/apache/flink/pull/2228 Thank you for rebasing again. I'm merging the change ... --- 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

[GitHub] flink issue #2228: [FLINK-4170][kinesis-connector] Simplify Kinesis connecte...

2016-07-13 Thread tzulitai
Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/2228 Rebased again on the new endpoint config merged in https://github.com/apache/flink/pull/2227. This PR should be ready for a final review now. --- If your project is set up for it, you can

[GitHub] flink issue #2228: [FLINK-4170][kinesis-connector] Simplify Kinesis connecte...

2016-07-13 Thread rmetzger
Github user rmetzger commented on the issue: https://github.com/apache/flink/pull/2228 I think there was an issue with Github and travis: https://www.traviscistatus.com/incidents/t4xn7bmq7fww --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] flink issue #2228: [FLINK-4170][kinesis-connector] Simplify Kinesis connecte...

2016-07-12 Thread tzulitai
Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/2228 Sorry, had a failing test after the rebase. Fixed. --- 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

[GitHub] flink issue #2228: [FLINK-4170][kinesis-connector] Simplify Kinesis connecte...

2016-07-12 Thread uce
Github user uce commented on the issue: https://github.com/apache/flink/pull/2228 I think this is an improvement over the current state. --- 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

[GitHub] flink issue #2228: [FLINK-4170][kinesis-connector] Simplify Kinesis connecte...

2016-07-12 Thread tzulitai
Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/2228 Rebasing on merge of https://github.com/apache/flink/pull/2071 .. --- 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

[GitHub] flink issue #2228: [FLINK-4170][kinesis-connector] Simplify Kinesis connecte...

2016-07-12 Thread tzulitai
Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/2228 We should probably also merge https://github.com/apache/flink/pull/2227 before this one, to reduce effort on rebasing. --- If your project is set up for it, you can reply to this email and have