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

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

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

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 preparePartitions

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

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

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

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 issue #1702: Update Eventhub-Client jar dependency to 1.0.1

Github user raviperi commented on the issue: https://github.com/apache/storm/pull/1702 The latest PR addresses the above comments . Move dependencies to parent pom . Revert changes to versions to reflect naming convention. I will be sending a different PR to master

[GitHub] storm issue #1717: Update eventhub client dependency. Move Storm-Eventhubs d...

Github user raviperi commented on the issue: https://github.com/apache/storm/pull/1717 Below JIRA that reflects the need for the above work. STORM-2127 - Storm-eventhubs should use latest amqp and eventhubs-client versions --- If your project is set up for it, you can reply

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

Github user raviperi commented on the issue: https://github.com/apache/storm/pull/1702 Created a new PR https://github.com/apache/storm/pull/1717 for the master branch. Created a new JIRA that reflects the need for the aboce work. STORM-2127 - Storm-eventhubs should use

[GitHub] storm pull request #1717: Update eventhub client dependency. Move Storm-Even...

GitHub user raviperi opened a pull request: https://github.com/apache/storm/pull/1717 Update eventhub client dependency. Move Storm-Eventhubs dependency ve… Update eventhub client dependency. Move Storm-Eventhubs dependency vesion definitions to parent pom. Introduce new schemes

[GitHub] storm pull request #1738: STORM-2127: Storm-eventhubs should use latest amqp...

GitHub user raviperi opened a pull request: https://github.com/apache/storm/pull/1738 STORM-2127: Storm-eventhubs should use latest amqp and eventhubs-client versions STORM-2127: Storm-eventhubs should use latest amqp and eventhubs-client versions You can merge this pull request

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

Github user raviperi commented on the issue: https://github.com/apache/storm/pull/1702 @revans2 Please let me know if you have further comments. --- 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

[GitHub] storm issue #1717: STORM-2127: Storm-eventhubs should use latest amqp and ev...

Github user raviperi commented on the issue: https://github.com/apache/storm/pull/1717 @harshach Updated title as per your recommendation. --- 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

[GitHub] storm issue #1702: STORM-2127: Storm-eventhubs should use latest amqp and ev...

Github user raviperi commented on the issue: https://github.com/apache/storm/pull/1702 @harshach Updated title as per your recommendation. --- 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

[GitHub] storm pull request #2322: Introduce config params to use latest EH client, c...

GitHub user raviperi opened a pull request: https://github.com/apache/storm/pull/2322 Introduce config params to use latest EH client, control request prefetch size, batch size of events received per call. -Refactor the code to group classes more appropriately -Remove