[jira] [Resolved] (STORM-2079) Unneccessary readStormConfig operation

[ https://issues.apache.org/jira/browse/STORM-2079?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim resolved STORM-2079. - Resolution: Fixed Fix Version/s: 1.0.3 1.1.0 2.0.0

[GitHub] storm pull request #1670: [STORM-2079] - Unneccessary readStormConfig operat...

Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1670 --- 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 not have this feature enabled and wishes so, or if the feature is

[jira] [Updated] (STORM-2074) NPE bug in storm-kafka-monitor

[ https://issues.apache.org/jira/browse/STORM-2074?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim updated STORM-2074: Component/s: storm-kafka-monitor > NPE bug in storm-kafka-monitor > --

[jira] [Resolved] (STORM-2074) NPE bug in storm-kafka-monitor

[ https://issues.apache.org/jira/browse/STORM-2074?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim resolved STORM-2074. - Resolution: Fixed Fix Version/s: 1.1.0 2.0.0 Thanks [~vesense], I

[GitHub] storm pull request #1665: STORM-2074: fix storm-kafka-monitor NPE bug

Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1665 --- 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 not have this feature enabled and wishes so, or if the feature is

[jira] [Resolved] (STORM-2081) create external directory for storm-sql various data sources and move storm-sql-kafka to it

[ https://issues.apache.org/jira/browse/STORM-2081?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim resolved STORM-2081. - Resolution: Fixed Fix Version/s: 1.0.3 1.1.0 2.0.0

[jira] [Updated] (STORM-2081) create external directory for storm-sql various data sources and move storm-sql-kafka to it

[ https://issues.apache.org/jira/browse/STORM-2081?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim updated STORM-2081: Component/s: storm-sql > create external directory for storm-sql various data sources and move >

[GitHub] storm pull request #1672: [STORM-2081][SQL] create external directory for st...

Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1672 --- 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 not have this feature enabled and wishes so, or if the feature is

Re: [DISCUSS] storm-jms Code Donation

+1 for getting this into Apache Storm. I looked at this project earlier and I am open to be a committer sponsor for storm-jms. Thanks, Satish. On Tue, Sep 6, 2016 at 7:15 AM, P. Taylor Goetz wrote: > I'm totally open to sponsoring/supporting it. The code is fairly stable >

Re: [DISCUSS] Accept JW Player SQE Code Donation

Agree with Jungtaek on the below. - Better to support SQL instead of SQL like (SQL like creates confusions). We are using Apache Calcite, we should continue with that. - Currently trident is used but we should move to windowing abstractions later for specifying boundedness to run the

[GitHub] storm issue #1665: STORM-2074: fix storm-kafka-monitor NPE bug

Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1665 +1 --- 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 not have this feature enabled and wishes so, or if the

[GitHub] storm issue #1672: [STORM-2081][SQL] create external directory for storm-sql...

Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1672 @vesense Thanks for following up this issue. +1 --- 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 not have this

[jira] [Resolved] (STORM-2059) storm-submit-tools is getting rat failures.

[ https://issues.apache.org/jira/browse/STORM-2059?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim resolved STORM-2059. - Resolution: Fixed Fix Version/s: 1.1.0 Fixed via STORM-2054 > storm-submit-tools is

[GitHub] storm issue #1673: STORM-2054 DependencyResolver should be aware of relative...

Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1673 There was merge conflict which is due to indentation off. --- 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 not

[jira] [Resolved] (STORM-2054) DependencyResolver should be aware of relative path and absolute path

[ https://issues.apache.org/jira/browse/STORM-2054?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim resolved STORM-2054. - Resolution: Fixed Fix Version/s: 1.1.0 2.0.0 Merged into master (by

[GitHub] storm pull request #1673: STORM-2054 DependencyResolver should be aware of r...

Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1673 --- 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 not have this feature enabled and wishes so, or if the feature is

[GitHub] storm pull request #1673: STORM-2054 DependencyResolver should be aware of r...

GitHub user HeartSaVioR opened a pull request: https://github.com/apache/storm/pull/1673 STORM-2054 DependencyResolver should be aware of relative path and absolute path (1.x) * Fix Booter to handle the path with awareness of relative vs absolute * Modify DependencyResolverTest

Re: [DISCUSS] storm-jms Code Donation

I'm totally open to sponsoring/supporting it. The code is fairly stable and hasn't changed much for years. The open pull requests are largely on hold pending a move to Apache. They are from new contributors that would potentially require an additional icla. Moving it to Apache would simplify

Re: [DISCUSS] Accept JW Player SQE Code Donation

Thanks JW Player folks to come in and express your support. I can see the sponsors of SQE which makes me feel that SQE is nice enough. And also I agree "production-ready" is a great point to value. I have been positive to merge this in, just wondering how we merge Storm SQL and SQE for exposing

Re: [DISCUSS] storm-jms Code Donation

This is good to merge since this is not duplicated / conflicted to anything on current Apache Storm project. I would be +1 if any committer/PMCs is/are supportive and add themselves to committer sponsor. There're a few issues and pull requests, and most of them are stalled (several things are

[GitHub] storm issue #1642: STORM-2018: Supervisor V2.

Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1642 Not sure what has been happening with travis not being able to get to the apache maven repo all the time, but my build in travis passed https://travis-ci.org/revans2/incubator-storm/builds/157725399

[jira] [Commented] (STORM-2054) DependencyResolver should be aware of relative path and absolute path

[ https://issues.apache.org/jira/browse/STORM-2054?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15465719#comment-15465719 ] Robert Joseph Evans commented on STORM-2054: I merged the pull request to master, but it looks

[GitHub] storm pull request #1648: STORM-2054 DependencyResolver should be aware of r...

Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1648 --- 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 not have this feature enabled and wishes so, or if the feature is

[GitHub] storm issue #1642: STORM-2018: Supervisor V2.

Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1642 I could not reproduce the one failure and the rat failure is fixed by STORM-2054 https://github.com/apache/storm/pull/1648 --- If your project is set up for it, you can reply to this email and

[GitHub] storm issue #1648: STORM-2054 DependencyResolver should be aware of relative...

Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1648 +1 fixes the rat issues on storm-submit-tools. --- 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 not have this

[GitHub] storm issue #1642: STORM-2018: Supervisor V2.

Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1642 Looks like there were two failures in travis. One is a rat issue with storm-submit-tools the other is an integration test that timed out. I'll try to reproduce the issues and see if I can fix

[GitHub] storm issue #1642: STORM-2018: Supervisor V2.

Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1642 I recovered it and fixed some issues with integration tests/rat. I think it should be good to go. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] storm issue #1642: STORM-2018: Supervisor V2.

Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1642 @srdo Ya that is what I did. Thanks for the advice. Not sure what happened somehow when I upmerged all but 2 of my commits disappeared. --- If your project is set up for it, you can reply to

[GitHub] storm issue #1642: STORM-2018: Supervisor V2.

Github user srdo commented on the issue: https://github.com/apache/storm/pull/1642 Check the ref log, maybe you can recover https://git-scm.com/docs/git-reflog --- 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

[GitHub] storm issue #1642: STORM-2018: Supervisor V2.

Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1642 Sorry, git just ate all of my more recent changes, and I have no idea what happened. I'll try to see if I can recover them, but I might have to start over... --- If your project is set up for it,

[GitHub] storm pull request #1642: STORM-2018: Supervisor V2.

Github user abellina commented on a diff in the pull request: https://github.com/apache/storm/pull/1642#discussion_r77532577 --- Diff: storm-core/src/jvm/org/apache/storm/localizer/LocalDownloadedResource.java --- @@ -0,0 +1,107 @@ +/** + * Licensed to the Apache Software

[GitHub] storm pull request #1642: STORM-2018: Supervisor V2.

Github user abellina commented on a diff in the pull request: https://github.com/apache/storm/pull/1642#discussion_r77532348 --- Diff: storm-core/src/jvm/org/apache/storm/daemon/supervisor/Slot.java --- @@ -0,0 +1,769 @@ +/** + * Licensed to the Apache Software Foundation

Storm Nullpointer exception

Hi, After running our storm cluster in AWS for few days , we are getting a Nullpointer exception in worker logs. Do you have any suggestions on this issue? 2016-09-05 02:53:25.120 o.a.s.m.n.StormServerHandler [ERROR] server errors in handling the request java.lang.NullPointerException at com.

[GitHub] storm issue #1642: STORM-2018: Supervisor V2.

Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1642 I just pushed in a much of new fixes and addressed all of the outstanding review comments. I think it is good to go. Please take another look/test it and let me know. I have run tests

[GitHub] storm pull request #1642: STORM-2018: Supervisor V2.

Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/1642#discussion_r77527879 --- Diff: storm-core/src/jvm/org/apache/storm/localizer/AsyncLocalizer.java --- @@ -0,0 +1,420 @@ +/** + * Licensed to the Apache Software Foundation

[GitHub] storm pull request #1642: STORM-2018: Supervisor V2.

Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/1642#discussion_r77527810 --- Diff: storm-core/src/jvm/org/apache/storm/localizer/AsyncLocalizer.java --- @@ -0,0 +1,420 @@ +/** + * Licensed to the Apache Software Foundation

[GitHub] storm pull request #1642: STORM-2018: Supervisor V2.

Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/1642#discussion_r77526779 --- Diff: storm-core/src/jvm/org/apache/storm/daemon/supervisor/Slot.java --- @@ -0,0 +1,766 @@ +/** + * Licensed to the Apache Software Foundation

[jira] [Created] (STORM-2082) add sql external module storm-sql-hdfs

Xin Wang created STORM-2082: --- Summary: add sql external module storm-sql-hdfs Key: STORM-2082 URL: https://issues.apache.org/jira/browse/STORM-2082 Project: Apache Storm Issue Type: Improvement

[GitHub] storm pull request #1642: STORM-2018: Supervisor V2.

Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/1642#discussion_r77526151 --- Diff: storm-core/src/jvm/org/apache/storm/daemon/supervisor/ReadClusterState.java --- @@ -0,0 +1,318 @@ +/** + * Licensed to the Apache Software