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
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 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 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 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 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 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 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 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 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 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 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 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 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
14 matches
Mail list logo