Github user ptgoetz commented on the issue:
https://github.com/apache/storm/pull/2203
Crude test, but illustrates the cost of meters:
(code marks a meter | increments a counter from 0 to `Integer.MAX`)
```
*** METER ***
Time: 126.39
ops/sec: 16,990,930
*** COUNTE
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/2430#discussion_r152434183
--- Diff:
storm-webapp/src/main/java/org/apache/storm/daemon/logviewer/handler/LogviewerLogSearchHandler.java
---
@@ -97,8 +98,14 @@ public LogviewerLog
Github user ptgoetz commented on the issue:
https://github.com/apache/storm/pull/2203
@revans2 Thanks for the update.
My first instinct is that itâs usage of Meter on the critical path.
Adding anything that âdoes somethingâ there is going to add some level of
overhead.
Github user Ethanlm commented on a diff in the pull request:
https://github.com/apache/storm/pull/2430#discussion_r152412106
--- Diff:
storm-webapp/src/main/java/org/apache/storm/daemon/logviewer/handler/LogviewerLogSearchHandler.java
---
@@ -97,8 +98,14 @@ public LogviewerLogSear
GitHub user Ethanlm opened a pull request:
https://github.com/apache/storm/pull/2432
[STORM-2829] fix logviewer deepSearch direct self-reference leading to
cycle exception
https://issues.apache.org/jira/browse/STORM-2829
```
2017-11-21 21:06:19.369 o.e.j.s.HttpChannel q
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2430#discussion_r152407885
--- Diff:
storm-webapp/src/main/java/org/apache/storm/daemon/logviewer/handler/LogviewerLogSearchHandler.java
---
@@ -97,8 +98,14 @@ public LogviewerLogSear
Github user Ethanlm commented on a diff in the pull request:
https://github.com/apache/storm/pull/2430#discussion_r152392265
--- Diff:
storm-webapp/src/main/java/org/apache/storm/daemon/logviewer/handler/LogviewerLogSearchHandler.java
---
@@ -97,8 +98,14 @@ public LogviewerLogSear
Github user srdo commented on the issue:
https://github.com/apache/storm/pull/2426
Thanks for the reviews.
---
Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/2426
---
Github user srdo commented on a diff in the pull request:
https://github.com/apache/storm/pull/2430#discussion_r152371328
--- Diff:
storm-webapp/src/main/java/org/apache/storm/daemon/logviewer/handler/LogviewerLogSearchHandler.java
---
@@ -97,8 +98,14 @@ public LogviewerLogSearchH
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2430#discussion_r152368944
--- Diff:
storm-webapp/src/main/java/org/apache/storm/daemon/logviewer/handler/LogviewerLogSearchHandler.java
---
@@ -97,8 +98,14 @@ public LogviewerLogSear
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2430#discussion_r152368599
--- Diff:
storm-webapp/src/main/java/org/apache/storm/daemon/logviewer/handler/LogviewerLogSearchHandler.java
---
@@ -97,8 +98,14 @@ public LogviewerLogSear
Github user srdo commented on a diff in the pull request:
https://github.com/apache/storm/pull/2430#discussion_r152365226
--- Diff:
storm-webapp/src/main/java/org/apache/storm/daemon/logviewer/handler/LogviewerLogSearchHandler.java
---
@@ -97,8 +98,14 @@ public LogviewerLogSearchH
Github user hmcl commented on the issue:
https://github.com/apache/storm/pull/2426
+1. Thanks @srdo. Can you please squash the commits before merging. Thanks.
---
Github user Ethanlm commented on a diff in the pull request:
https://github.com/apache/storm/pull/2430#discussion_r152360979
--- Diff:
storm-webapp/src/main/java/org/apache/storm/daemon/logviewer/handler/LogviewerLogSearchHandler.java
---
@@ -97,8 +98,14 @@ public LogviewerLogSear
Github user srdo commented on the issue:
https://github.com/apache/storm/pull/2430
I think some of the tests are still flaky.
---
Github user srdo commented on a diff in the pull request:
https://github.com/apache/storm/pull/2430#discussion_r152348839
--- Diff:
storm-webapp/src/main/java/org/apache/storm/daemon/logviewer/handler/LogviewerLogSearchHandler.java
---
@@ -97,8 +98,14 @@ public LogviewerLogSearchH
Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/2203
Also looking at the numbers from @HeartSaVioR. They also show the CPU
being higher, but not nearly as much as mine do 219 seconds for V2 vs 206
seconds for 1.2.0 or about a 6% difference.
A
Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/2203
Here are my numbers, and no I didn't do it headless but I was not doing
anything on the laptop at the time it was running except looking at Activity
Monitor to be sure nothing else was using too much
Github user Ethanlm commented on the issue:
https://github.com/apache/storm/pull/2430
The travis test passed on my repo:
https://travis-ci.org/Ethanlm/storm/builds/304993981
Don't know why it failed here.
---
Hi All,
In my project we already installed and running Storm and it’s components as
process but we are planning to install/run storm and it’s components as
services.
Could you please help what all things needs to be taken care which performing
this activity.
Regards
Srikanth Vattikuti
Hi All,
In my project we already installed and running Storm and it’s components as
process but we are planning to install/run storm and it’s components as
services.
Could you please help what all things needs to be taken care which performing
this activity.
Regards
Srikanth Vattikuti
Alexandre,
It's a bug in the way I tried to fix the NPE you had a few days ago in
https://github.com/apache/storm/pull/2428. I missed that using
setKey/setValue actually builds a new KafkaSpoutConfig.Builder instead of
just setting a field, and the change I made to the copy constructor means
that
Github user danny0405 commented on the issue:
https://github.com/apache/storm/pull/2319
@HeartSaVioR i have made a PR for storm 2.0, really thx for your review
work, this is the link:
https://github.com/apache/storm/pull/2431/
---
Github user danny0405 commented on the issue:
https://github.com/apache/storm/pull/2431
@HeartSaVioR hi, i have make PR for storm2.0, please help me to review this
---
Hello Stig,
Here's an update of my tests with storm 1.2.0 preview:
- I accept the limitation on the stability of the string format returned by
getTopicsString(), as I have adapted our code to detect both 1.1.0-style &
1.2.0-style. Isn't there a clean way to get the list of topics other than
our fr
GitHub user danny0405 opened a pull request:
https://github.com/apache/storm/pull/2431
heartbeats and assignments promotion for storm2.0
For large cluster support, good scheduling performance for scheduling and
assignments distribution.
This is PR for 1.1.x-branch of heartb
27 matches
Mail list logo