Github user raghavgautam commented on the issue:
https://github.com/apache/storm/pull/2660
@ghajos Ping.
---
Github user zd-project commented on a diff in the pull request:
https://github.com/apache/storm/pull/2714#discussion_r195246920
--- Diff:
storm-server/src/main/java/org/apache/storm/metric/StormMetricsRegistry.java ---
@@ -73,6 +130,12 @@ private static void
startMetricsReporter(P
Github user agresch commented on a diff in the pull request:
https://github.com/apache/storm/pull/2714#discussion_r195243886
--- Diff:
storm-server/src/main/java/org/apache/storm/metric/StormMetricsRegistry.java ---
@@ -73,6 +130,12 @@ private static void
startMetricsReporter(Prep
GitHub user agresch opened a pull request:
https://github.com/apache/storm/pull/2718
STORM-3103 allow nimbus to shutdown properly
Nimbus registers exitProcess() for the StormTimer.onKill routine. This
does not return once called. This basically locks up the timer with active
bein
Github user srdo commented on the issue:
https://github.com/apache/storm/pull/2715
+1
---
Github user srdo commented on the issue:
https://github.com/apache/storm/pull/2717
+1. Please squash to one commit.
---
Github user srdo commented on the issue:
https://github.com/apache/storm/pull/2716
+1
---
GitHub user acseidel opened a pull request:
https://github.com/apache/storm/pull/2716
STORM-3102 Remove check for partition offset before every emit.
In kafka >0.10.2 this check became expensive, causing large performance
decreases.
You can merge this pull request into a Git rep
GitHub user acseidel opened a pull request:
https://github.com/apache/storm/pull/2717
Bugfix/storm 3102 kafka client performance storm 1.x
In kafka >0.10.2 this check became expensive, causing large performance
decreases.
You can merge this pull request into a Git repository by
GitHub user acseidel opened a pull request:
https://github.com/apache/storm/pull/2715
STORM-3102 Remove check for partition offset before every emit.
IN kafka >0.10.2 this check became expensive, causing large performance
decreases.
You can merge this pull request into a Git rep
Github user MaurGi commented on a diff in the pull request:
https://github.com/apache/storm/pull/2712#discussion_r195140675
--- Diff: docs/Multilang-protocol.md ---
@@ -7,6 +7,18 @@ This page explains the multilang protocol as of Storm
0.7.1. Versions prior to 0
# Storm
Github user MaurGi commented on the issue:
https://github.com/apache/storm/pull/2712
thanks for the feedback, modified and resubmitted -
---
Github user zd-project commented on the issue:
https://github.com/apache/storm/pull/2714
Some questions:
1. Currently, by rejecting a metric, StormMetricsRegistry simply discard it
and log the warning without throwing any errors to the calling method/classes.
Is this an appropriat
GitHub user zd-project opened a pull request:
https://github.com/apache/storm/pull/2714
STORM-3101: Add filter to StormMetricsRegistry.
The current implementation is based on the idea to set a source for a
certain process. All the metrics not belonging to the source (e.g., superviso
GitHub user ManoharVanam opened a pull request:
https://github.com/apache/storm/pull/2713
[STORM-3094] : Added topology name validation at client side
Current Behavior : Execute topology with invalid topology name is throwing
exception after uploading the jar.
Improvement :
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/2712#discussion_r195078534
--- Diff: docs/Multilang-protocol.md ---
@@ -7,6 +7,18 @@ This page explains the multilang protocol as of Storm
0.7.1. Versions prior to 0
# S
Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/2703
---
Github user danny0405 commented on the issue:
https://github.com/apache/storm/pull/2636
@cfriaszapater
Ok, look forward to your master branch patch.
---
Github user cfriaszapater commented on the issue:
https://github.com/apache/storm/pull/2636
Hi danny0405,
In response to HeartSaVioR, I said:
> I'll try to do that pull request in the future to master branch.
> I did this to 1.1.x branch because we use it in our organiz
Github user cfriaszapater closed the pull request at:
https://github.com/apache/storm/pull/2636
---
20 matches
Mail list logo
