[GitHub] storm issue #2590: STORM-2974: Add transactional non-opaque spout to storm-k...

2018-07-13 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/2590 I have found that couple of public classes are renamed, but they're in `internal` package which makes others feeling non-public, so I think we are OK. ---

[GitHub] storm issue #2590: STORM-2974: Add transactional non-opaque spout to storm-k...

2018-07-10 Thread srdo
Github user srdo commented on the issue: https://github.com/apache/storm/pull/2590 @HeartSaVioR Rebased. ---

[GitHub] storm issue #2590: STORM-2974: Add transactional non-opaque spout to storm-k...

2018-07-09 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/2590 @srdo I'm sorry but could you please rebase this? Thanks in advance! ---

[GitHub] storm issue #2590: STORM-2974: Add transactional non-opaque spout to storm-k...

2018-03-10 Thread srdo
Github user srdo commented on the issue: https://github.com/apache/storm/pull/2590 I have tested the non-opaque spout by running the storm-kafka-client-examples topology, so testing hasn't been very extensive. ---