[GitHub] storm pull request #1702: Update Eventhub-Client jar dependency to 1.0.1

2016-10-03 Thread raviperi
Github user raviperi commented on a diff in the pull request: https://github.com/apache/storm/pull/1702#discussion_r81616391 --- Diff: external/storm-eventhubs/src/main/java/org/apache/storm/eventhubs/spout/StringEventDataScheme.java --- @@ -0,0 +1,69 @@

[GitHub] storm pull request #1702: Update Eventhub-Client jar dependency to 1.0.1

2016-10-03 Thread raviperi
Github user raviperi commented on a diff in the pull request: https://github.com/apache/storm/pull/1702#discussion_r81605840 --- Diff: external/storm-eventhubs/src/main/java/org/apache/storm/eventhubs/spout/EventHubSpout.java --- @@ -121,8 +121,8 @@ public void

[GitHub] storm pull request #1702: Update Eventhub-Client jar dependency to 1.0.1

2016-09-27 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/1702#discussion_r80776662 --- Diff: external/storm-eventhubs/src/main/java/org/apache/storm/eventhubs/spout/EventHubSpout.java --- @@ -152,7 +152,7 @@ public void open(Map config,

[GitHub] storm pull request #1702: Update Eventhub-Client jar dependency to 1.0.1

2016-09-27 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/1702#discussion_r80775363 --- Diff: external/storm-eventhubs/src/main/java/org/apache/storm/eventhubs/spout/EventHubSpout.java --- @@ -152,7 +152,7 @@ public void open(Map config,

[GitHub] storm pull request #1702: Update Eventhub-Client jar dependency to 1.0.1

2016-09-27 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/1702#discussion_r80771322 --- Diff: external/storm-eventhubs/src/main/java/org/apache/storm/eventhubs/spout/EventHubSpout.java --- @@ -152,7 +152,7 @@ public void open(Map config,

[GitHub] storm pull request #1702: Update Eventhub-Client jar dependency to 1.0.1

2016-09-27 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/1702#discussion_r80771003 --- Diff: external/storm-eventhubs/src/main/java/org/apache/storm/eventhubs/spout/StringEventDataScheme.java --- @@ -0,0 +1,69 @@

[GitHub] storm pull request #1702: Update Eventhub-Client jar dependency to 1.0.1

2016-09-27 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/1702#discussion_r80770524 --- Diff: external/storm-eventhubs/pom.xml --- @@ -1,113 +1,120 @@

[GitHub] storm pull request #1702: Update Eventhub-Client jar dependency to 1.0.1

2016-09-27 Thread raviperi
Github user raviperi commented on a diff in the pull request: https://github.com/apache/storm/pull/1702#discussion_r80753947 --- Diff: external/storm-eventhubs/pom.xml --- @@ -21,19 +21,19 @@ storm org.apache.storm -1.0.3-SNAPSHOT

[GitHub] storm pull request #1702: Update Eventhub-Client jar dependency to 1.0.1

2016-09-27 Thread raviperi
Github user raviperi commented on a diff in the pull request: https://github.com/apache/storm/pull/1702#discussion_r80753320 --- Diff: external/storm-eventhubs/pom.xml --- @@ -21,19 +21,19 @@ storm org.apache.storm -1.0.3-SNAPSHOT

[GitHub] storm pull request #1702: Update Eventhub-Client jar dependency to 1.0.1

2016-09-27 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/1702#discussion_r80720194 --- Diff: external/storm-eventhubs/pom.xml --- @@ -21,19 +21,19 @@ storm org.apache.storm -1.0.3-SNAPSHOT

[GitHub] storm pull request #1702: Update Eventhub-Client jar dependency to 1.0.1

2016-09-22 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/1702#discussion_r80133198 --- Diff: external/storm-eventhubs/pom.xml --- @@ -21,19 +21,19 @@ storm org.apache.storm -1.0.3-SNAPSHOT

[GitHub] storm pull request #1702: Update Eventhub-Client jar dependency to 1.0.1

2016-09-22 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/1702#discussion_r80133790 --- Diff: external/storm-eventhubs/pom.xml --- @@ -21,19 +21,19 @@ storm org.apache.storm -1.0.3-SNAPSHOT

[GitHub] storm pull request #1702: Update Eventhub-Client jar dependency to 1.0.1

2016-09-22 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/1702#discussion_r79990064 --- Diff: external/storm-eventhubs/pom.xml --- @@ -21,19 +21,19 @@ storm org.apache.storm -1.0.3-SNAPSHOT +

[GitHub] storm pull request #1702: Update Eventhub-Client jar dependency to 1.0.1

2016-09-21 Thread raviperi
GitHub user raviperi opened a pull request: https://github.com/apache/storm/pull/1702 Update Eventhub-Client jar dependency to 1.0.1 Eventhub client library has few key fixes that need to be included in storm-eventhub jar. Fixes: 1. Fix for EventHubSender may throw