[GitHub] markusthoemmes commented on issue #3072: Enhance kafka message provider

2018-02-21 Thread GitBox
markusthoemmes commented on issue #3072: Enhance kafka message provider URL: https://github.com/apache/incubator-openwhisk/pull/3072#issuecomment-367330648 PG3 1895 ? This is an automated message from the Apache Git

[GitHub] markusthoemmes commented on issue #3072: Enhance kafka message provider

2018-02-19 Thread GitBox
markusthoemmes commented on issue #3072: Enhance kafka message provider URL: https://github.com/apache/incubator-openwhisk/pull/3072#issuecomment-366713038 I clarified the remaining bits with Kafka experts. Consider this scheduled for merge after #3258 to minimize conflicts. @devbv

[GitHub] markusthoemmes commented on issue #3072: Enhance kafka message provider

2018-01-17 Thread GitBox
markusthoemmes commented on issue #3072: Enhance kafka message provider URL: https://github.com/apache/incubator-openwhisk/pull/3072#issuecomment-358226564 @rabbah can you please sanity check this? This is an automated

[GitHub] markusthoemmes commented on issue #3072: Enhance kafka message provider

2017-12-07 Thread GitBox
markusthoemmes commented on issue #3072: Enhance kafka message provider URL: https://github.com/apache/incubator-openwhisk/pull/3072#issuecomment-350013610 Okay, that makes sense. The major difference from the better-known clients is, that their error behavior is more principled

[GitHub] markusthoemmes commented on issue #3072: Enhance kafka message provider

2017-12-07 Thread GitBox
markusthoemmes commented on issue #3072: Enhance kafka message provider URL: https://github.com/apache/incubator-openwhisk/pull/3072#issuecomment-349955038 Can you provide tests for the failure scenarios you implemented here? In general I have the feeling that we're moving into a