[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-08 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r208577649 --- Diff: external/storm-kafka-client/src/main/java/org/apache/storm/kafka/bolt/PreparableCallback.java --- @@ -0,0 +1,32 @@ +/** + * Licensed to the

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-07 Thread danny0405
Github user danny0405 commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207771586 --- Diff: external/storm-kafka-client/src/main/java/org/apache/storm/kafka/bolt/PreparableCallback.java --- @@ -0,0 +1,32 @@ +/** + * Licensed to

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/2790 ---

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-05 Thread dfdemar
Github user dfdemar commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207761956 --- Diff: external/storm-kafka-client/src/test/java/org/apache/storm/kafka/bolt/KafkaBoltTest.java --- @@ -17,73 +17,140 @@ */ package

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-04 Thread dfdemar
Github user dfdemar commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207711135 --- Diff: external/storm-kafka-client/src/test/java/org/apache/storm/kafka/bolt/KafkaBoltTest.java --- @@ -17,73 +17,140 @@ */ package

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-04 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207709023 --- Diff: external/storm-kafka-client/src/test/java/org/apache/storm/kafka/bolt/KafkaBoltTest.java --- @@ -17,73 +17,140 @@ */ package

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-04 Thread dfdemar
Github user dfdemar commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207708527 --- Diff: external/storm-kafka-client/src/test/java/org/apache/storm/kafka/bolt/KafkaBoltTest.java --- @@ -17,73 +17,140 @@ */ package

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-04 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207706595 --- Diff: external/storm-kafka-client/src/test/java/org/apache/storm/kafka/bolt/KafkaBoltTest.java --- @@ -17,73 +17,140 @@ */ package

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-03 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207671937 --- Diff: external/storm-kafka-client/src/main/java/org/apache/storm/kafka/bolt/KafkaBolt.java --- @@ -62,6 +62,7 @@ private OutputCollector

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-03 Thread dfdemar
Github user dfdemar commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207649108 --- Diff: external/storm-kafka-client/src/main/java/org/apache/storm/kafka/bolt/KafkaBolt.java --- @@ -62,6 +62,7 @@ private OutputCollector

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-03 Thread dfdemar
Github user dfdemar commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207560866 --- Diff: external/storm-kafka-client/src/main/java/org/apache/storm/kafka/bolt/KafkaBolt.java --- @@ -129,6 +140,27 @@ public void prepare(Map topoConf,

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-03 Thread dfdemar
Github user dfdemar commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207537361 --- Diff: external/storm-kafka-client/src/main/java/org/apache/storm/kafka/bolt/KafkaBolt.java --- @@ -62,6 +62,7 @@ private OutputCollector

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-03 Thread dfdemar
Github user dfdemar commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207534991 --- Diff: external/storm-kafka-client/src/main/java/org/apache/storm/kafka/bolt/KafkaBolt.java --- @@ -129,6 +140,27 @@ public void prepare(Map topoConf,

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-03 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207526335 --- Diff: external/storm-kafka-client/src/test/java/org/apache/storm/kafka/bolt/KafkaBoltTest.java --- @@ -37,38 +39,48 @@ import

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-03 Thread dfdemar
Github user dfdemar commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207524658 --- Diff: external/storm-kafka-client/src/test/java/org/apache/storm/kafka/bolt/KafkaBoltTest.java --- @@ -37,38 +39,48 @@ import

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-03 Thread dfdemar
Github user dfdemar commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207524275 --- Diff: external/storm-kafka-client/src/test/java/org/apache/storm/kafka/bolt/KafkaBoltTest.java --- @@ -37,38 +39,48 @@ import

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-03 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207511238 --- Diff: external/storm-kafka-client/src/main/java/org/apache/storm/kafka/bolt/KafkaBolt.java --- @@ -62,6 +62,7 @@ private OutputCollector

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-03 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207515186 --- Diff: external/storm-kafka-client/src/test/java/org/apache/storm/kafka/bolt/KafkaBoltTest.java --- @@ -37,38 +39,48 @@ import

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-03 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207512766 --- Diff: external/storm-kafka-client/src/main/java/org/apache/storm/kafka/bolt/KafkaBolt.java --- @@ -62,6 +62,7 @@ private OutputCollector

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-03 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207514281 --- Diff: external/storm-kafka-client/src/test/java/org/apache/storm/kafka/bolt/KafkaBoltTest.java --- @@ -37,38 +39,48 @@ import

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-03 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207511912 --- Diff: external/storm-kafka-client/src/main/java/org/apache/storm/kafka/bolt/KafkaBolt.java --- @@ -129,6 +140,27 @@ public void prepare(Map topoConf,

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-02 Thread dfdemar
Github user dfdemar commented on a diff in the pull request: https://github.com/apache/storm/pull/2790#discussion_r207428218 --- Diff: external/storm-kafka-client/src/main/java/org/apache/storm/kafka/bolt/KafkaBolt.java --- @@ -129,6 +140,27 @@ public void prepare(Map topoConf,

[GitHub] storm pull request #2790: STORM-3175 - Allow usage of custom Callback.

2018-08-02 Thread dfdemar
GitHub user dfdemar opened a pull request: https://github.com/apache/storm/pull/2790 STORM-3175 - Allow usage of custom Callback. You can merge this pull request into a Git repository by running: $ git pull https://github.com/dfdemar/storm custom-kafkabolt-callback