[GitHub] storm issue #1597: STORM-1285 port backtype.storm.command.get-errors to java

2016-07-30 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1597 @abhishekagarwal87 @satishd Thanks for the review. I addressed all of comments. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[jira] [Commented] (STORM-1285) port backtype.storm.command.get-errors to java

2016-07-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1285?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15400991#comment-15400991 ] ASF GitHub Bot commented on STORM-1285: --- Github user HeartSaVioR commented on the is

Re: Question about Storm UI

2016-07-30 Thread Jungtaek Lim
Hi Boyang, AFAIK, there's no discussion on Storm UI yet. Could you share your sketched UI for this? And is it on top of STORM-1994 ? I mean what Alessandro submitted. A random thought from myself is that we shows too many numbers even topology inf

[jira] [Commented] (STORM-1285) port backtype.storm.command.get-errors to java

2016-07-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1285?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15400984#comment-15400984 ] ASF GitHub Bot commented on STORM-1285: --- Github user HeartSaVioR commented on a diff

[GitHub] storm pull request #1597: STORM-1285 port backtype.storm.command.get-errors ...

2016-07-30 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/1597#discussion_r72898640 --- Diff: storm-core/src/jvm/org/apache/storm/command/GetErrors.java --- @@ -0,0 +1,80 @@ +/** + * Licensed to the Apache Software Foundation (ASF)

[jira] [Commented] (STORM-1435) Build a single jar with dependency for StormSQL dependency

2016-07-30 Thread Haohui Mai (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1435?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15400926#comment-15400926 ] Haohui Mai commented on STORM-1435: --- The core jar essentially compiles the SQL statement

[jira] [Commented] (STORM-1038) Upgrade netty transport from 3.x to 4.x

2016-07-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15400888#comment-15400888 ] ASF GitHub Bot commented on STORM-1038: --- Github user hsun-cnnxty commented on the is

[GitHub] storm issue #1591: STORM-1038: Upgrade netty to 4.x in 1.x-branch

2016-07-30 Thread hsun-cnnxty
Github user hsun-cnnxty commented on the issue: https://github.com/apache/storm/pull/1591 Seems the Travis CI build will fail at random places which are not related to this PR. For example the last build failed due to `[ERROR] Failed to execute goal org.apache.maven.

[jira] [Commented] (STORM-1038) Upgrade netty transport from 3.x to 4.x

2016-07-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15400887#comment-15400887 ] ASF GitHub Bot commented on STORM-1038: --- Github user hsun-cnnxty commented on the is

[GitHub] storm issue #728: [STORM-1038] Upgraded netty to 4.x

2016-07-30 Thread hsun-cnnxty
Github user hsun-cnnxty commented on the issue: https://github.com/apache/storm/pull/728 @HeartSaVioR @harshach I posted performance test results on #1591. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If you

[jira] [Commented] (STORM-1038) Upgrade netty transport from 3.x to 4.x

2016-07-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15400885#comment-15400885 ] ASF GitHub Bot commented on STORM-1038: --- Github user hsun-cnnxty commented on the is

[GitHub] storm issue #1591: STORM-1038: Upgrade netty to 4.x in 1.x-branch

2016-07-30 Thread hsun-cnnxty
Github user hsun-cnnxty commented on the issue: https://github.com/apache/storm/pull/1591 Performance test results are attached here. [perf_compare_netty_3vs4_1.x-branch.zip](https://github.com/apache/storm/files/392165/perf_compare_netty_3vs4_1.x-branch.zip) Baseline: c