[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14325251#comment-14325251 ] ASF GitHub Bot commented on STORM-487: -- Github user harshach commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-74791899 @HeartSaVioR @dashengju I took care of it as part of this PR https://github.com/apache/storm/pull/434 . Thanks. > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans >Assignee: Jungtaek Lim > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14321448#comment-14321448 ] ASF GitHub Bot commented on STORM-487: -- Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-74376482 @dashengju @harshach Since there's no exec on Windows, we can check os, and use sub.call() if it's Windows, and use os.execvp() if it's not Windows. If you think it makes sense, I'll file new JIRA and post a relevant PR. Thanks! > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans >Assignee: Jungtaek Lim > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14304681#comment-14304681 ] ASF GitHub Bot commented on STORM-487: -- Github user dashengju commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-72795383 @harshach , ok, you or @HeartSaVioR should responsible for fix this bug, because I just know how to fix it in linux, but not in windows. thanks > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans >Assignee: Jungtaek Lim > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14304676#comment-14304676 ] ASF GitHub Bot commented on STORM-487: -- Github user harshach commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-72794546 @dashengju I second that ,just recently noticed it . We can file a follow-up jira for it. > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans >Assignee: Jungtaek Lim > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14304668#comment-14304668 ] ASF GitHub Bot commented on STORM-487: -- Github user dashengju commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-72793961 @harshach @HeartSaVioR @revans2 , this PR has produce a new bug in linux OS. In function exec_storm_class,line 181: -os.execvp(JAVA_CMD, all_args) # replaces the current process and -# never returns +# handling whitespaces in JAVA_CMD +sub.call(all_args) The origin code execute JAVA_CMD in exec style(replaces the current process and never returns), But the new code execute JAVA_CMD in sub process. Usually, nimbus/supervisor is running by daemon tools, so when nimbus/supervisor run in sub process, daemon tools can not stop the nimbus/supervisor process. > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans >Assignee: Jungtaek Lim > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14259233#comment-14259233 ] HARSHA BALASUBRAMANIAN commented on STORM-487: -- Hi, I tried this command in apache-storm-0,9.3 and got the following error. If I just run 'storm list' or 'storm nimbus', the command works fine. Should I do anything special to run the storm client in python. C:\storm\apache-storm-0.9.3\apache-storm-0.9.3\bin>c:\python\python storm list ** The storm client can only be run from within a release. You appear to be trying to run the client from a checkout of Storm's source code. You can download a Storm release at http://storm-project.net/downloads.html ** > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14255923#comment-14255923 ] ASF GitHub Bot commented on STORM-487: -- Github user harshach commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-67860921 Thanks @HeartSaVioR merged PR. > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14255916#comment-14255916 ] ASF GitHub Bot commented on STORM-487: -- Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/280 > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=1426#comment-1426 ] ASF GitHub Bot commented on STORM-487: -- Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-67813766 Should this be pluggable ? storm-master/bin/storm Python Script storm-master/external/storm-cli --- Proivide Bash Shell / Windows cmd Shell or any other launching script > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14255334#comment-14255334 ] ASF GitHub Bot commented on STORM-487: -- Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-67792351 @harshach Otherwise we can try to change storm.cmd to call bin\storm. I heard storm.cmd can register storm to Windows Service, so we could achieve it by modifying storm.cmd. @ChitturiPadma Yes, it will add prerequisite. It could be bothering for users, but it could save storm dev. to remove sth. that runs only Windows. As you can see, Storm committers doesn't use Windows. > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14255096#comment-14255096 ] ASF GitHub Bot commented on STORM-487: -- Github user ChitturiPadma commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-67764141 @HeartSavioR, if it works on distributed mode in windows cluster without any exceptions, you cab go ahead and merge it. But the users on windows should be given prerequisite to install Python. > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14254847#comment-14254847 ] ASF GitHub Bot commented on STORM-487: -- Github user harshach commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-67741881 follow-up feedback. In windows shebang line doesn't work so the users have to do python bin\storm nimbus etc.. . This can be solved by add .py extension to bin\storm . We can do this in a separate JIRA. @HeartSaVioR I'll do the merge. Thanks. > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14254730#comment-14254730 ] ASF GitHub Bot commented on STORM-487: -- Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-67735825 @harshach Thanks for reviewing! :) @ptgoetz @revans2 Since it would be tested from @ChitturiPadma and @harshach, and two +1 from committers, so we can merge it. :D > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14254247#comment-14254247 ] ASF GitHub Bot commented on STORM-487: -- Github user harshach commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-67712765 @HeartSaVioR sorry about taking so long to review this. Its been a challenge to get a windows box :). I tested this both on windows 8 and centos6.5 I tested all the commands in storm.py everything works as expected. I am +1 on merging this. Thanks. > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14236596#comment-14236596 ] ASF GitHub Bot commented on STORM-487: -- Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-65885963 Could anybody check/review this? > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14236595#comment-14236595 ] Jungtaek Lim commented on STORM-487: Could anybody check/review this? > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14200187#comment-14200187 ] ASF GitHub Bot commented on STORM-487: -- Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-61983313 Sorry it took me so long to respond. The changes look good to me, but I don't have a windows box so I cannot test the windows support, I am +1 on the changes so far, but would like to see someone test this on windows too. > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14199687#comment-14199687 ] ASF GitHub Bot commented on STORM-487: -- Github user ptgoetz commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-61919968 @HeartSaVioR sorry for the delay. I will test and reply when I have the chance. I would encourage others to do the same as well. > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14199470#comment-14199470 ] ASF GitHub Bot commented on STORM-487: -- Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-61908678 @revans2 @ptgoetz Hello. Any updates on it? From some JIRA issues we already talked that it will be better to have 'unified' storm script. So when my patch works well, we reduce great maintenance cost. And it may can resolve STORM-322, too. :) > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14160288#comment-14160288 ] ASF GitHub Bot commented on STORM-487: -- Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-58017099 @ChitturiPadma Oh, that means PR can resolve STORM-487. Btw, I'm using apache-0.9.2-incubating. Actually I don't use Windows (I use OSX), but I'm just willing to contribute Storm Project and STORM-487 is labeled 'newbie'. > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14160280#comment-14160280 ] ASF GitHub Bot commented on STORM-487: -- Github user ChitturiPadma commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-58015873 I tested with apache-0.9.2-incubating version, copying the changes. It worked fine in pseduo distributed mode. Which version of storm have u been using... Earlier in storm-0.9.0.1 version the error of not able to delete the file is thrown from FileUtils.java in commons-io. > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14159499#comment-14159499 ] ASF GitHub Bot commented on STORM-487: -- Github user ChitturiPadma commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-57932225 When i worked on storm-0.9.0-rc2, the issue existed only with modified python script that works on windows. storm.cmd doesnt show have this issue in file deletion. Will recheck with latest storm-0.9.2 release and will get back to you. > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14159481#comment-14159481 ] ASF GitHub Bot commented on STORM-487: -- Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-57930437 @ChitturiPadma I've just test killing topology using Storm 0.9.2. There're some warning but supervisor seems to not killed. It seems to have an issue you mentioned (about file deletion) but workers completely killed. I can re-submit topology and it's running fine. ``` 2014-10-05 17:48:50 b.s.d.supervisor [INFO] Shutting down 95111631-8f7f-46ed-a93a-8db1d01ae4e0:154eaf07-90ed-4f84-ad84-3b3b4eb51444 2014-10-05 17:48:53 b.s.d.supervisor [INFO] Shutting down and clearing state for id 154eaf07-90ed-4f84-ad84-3b3b4eb51444. Current supervisor time: 1412498932. State: :disallowed, Heartbeat: #backtype.storm.daemon.common.WorkerHeartbeat{:time-secs 1412498932, :storm-id "production-topology-1-1412498729", :executors #{[2 2] [5 5] [8 8] [11 11] [14 14] [17 17] [20 20] [23 23] [26 26] [-1 -1]}, :port 6701} 2014-10-05 17:48:53 b.s.d.supervisor [INFO] Shutting down 95111631-8f7f-46ed-a93a-8db1d01ae4e0:154eaf07-90ed-4f84-ad84-3b3b4eb51444 2014-10-05 17:48:59 b.s.d.supervisor [INFO] Shut down 95111631-8f7f-46ed-a93a-8db1d01ae4e0:154eaf07-90ed-4f84-ad84-3b3b4eb51444 2014-10-05 17:48:59 b.s.d.supervisor [INFO] Shutting down 95111631-8f7f-46ed-a93a-8db1d01ae4e0:816a0fd6-7cdf-4312-a5ac-758f7f55ce46 2014-10-05 17:48:59 b.s.util [INFO] Error when trying to kill 4408. Process is probably already dead. 2014-10-05 17:49:03 b.s.util [INFO] Error when trying to kill 4420. Process is probably already dead. 2014-10-05 17:49:03 b.s.d.supervisor [INFO] Shut down 95111631-8f7f-46ed-a93a-8db1d01ae4e0:154eaf07-90ed-4f84-ad84-3b3b4eb51444 2014-10-05 17:49:03 b.s.d.supervisor [INFO] Shutting down and clearing state for id 816a0fd6-7cdf-4312-a5ac-758f7f55ce46. Current supervisor time: 1412498932. State: :disallowed, Heartbeat: #backtype.storm.daemon.common.WorkerHeartbeat{:time-secs 1412498932, :storm-id "production-topology-1-1412498729", :executors #{[3 3] [6 6] [9 9] [12 12] [15 15] [18 18] [21 21] [24 24] [27 27] [-1 -1]}, :port 6702} 2014-10-05 17:49:03 b.s.d.supervisor [INFO] Shutting down 95111631-8f7f-46ed-a93a-8db1d01ae4e0:816a0fd6-7cdf-4312-a5ac-758f7f55ce46 2014-10-05 17:49:04 b.s.d.supervisor [WARN] Failed to cleanup worker 816a0fd6-7cdf-4312-a5ac-758f7f55ce46. Will retry later # 2014-10-05 17:49:04 b.s.d.supervisor [INFO] Shut down 95111631-8f7f-46ed-a93a-8db1d01ae4e0:816a0fd6-7cdf-4312-a5ac-758f7f55ce46 2014-10-05 17:49:04 b.s.d.supervisor [INFO] Shutting down and clearing state for id c26b2e31-d006-4d4d-9e56-45ca9391018d. Current supervisor time: 1412498932. State: :disallowed, Heartbeat: #backtype.storm.daemon.common.WorkerHeartbeat{:time-secs 1412498932, :storm-id "production-topology-1-1412498729", :executors #{[4 4] [7 7] [10 10] [13 13] [16 16] [19 19] [22 22] [25 25] [28 28] [-1 -1] [1 1]}, :port 6700} 2014-10-05 17:49:04 b.s.d.supervisor [INFO] Shutting down 95111631-8f7f-46ed-a93a-8db1d01ae4e0:c26b2e31-d006-4d4d-9e56-45ca9391018d 2014-10-05 17:49:04 b.s.d.supervisor [WARN] Failed to cleanup worker 816a0fd6-7cdf-4312-a5ac-758f7f55ce46. Will retry later # 2014-10-05 17:49:04 b.s.d.supervisor [INFO] Shut down 95111631-8f7f-46ed-a93a-8db1d01ae4e0:816a0fd6-7cdf-4312-a5ac-758f7f55ce46 2014-10-05 17:49:04 b.s.d.supervisor [INFO] Shutting down 95111631-8f7f-46ed-a93a-8db1d01ae4e0:c26b2e31-d006-4d4d-9e56-45ca9391018d 2014-10-05 17:49:04 b.s.util [INFO] Error when trying to kill 1536. Process is probably already dead. 2014-10-05 17:49:06 b.s.util [INFO] Error when trying to kill 3408. Process is probably already dead. 2014-10-05 17:49:06 b.s.util [INFO] Error when trying to kill 3464. Process is probably already dead. 2014-10-05 17:49:06 b.s.d.supervisor [WARN] Failed to cleanup worker c26b2e31-d006-4d4d-9e56-45ca9391018d. Will retry later # 2014-10-05 17:49:06 b.s.d.supervisor [INFO] Shut down 95111631-8f7f-46ed-a93a-8db1d01ae4e0:c26b2e31-d006-4d4d-9e56-45ca9391018d 2014-10-05 17:49:06 b.s.d.supervisor [WARN] Failed to cleanup worker c26b2e31-d006-4d4d-9e56-45ca9391018d. Will retry later # 2014-10-05 17:49:06 b.s.d.supervisor [INFO] Shut down 95111631-8f7f-46ed-a93a-8db1d01ae4e0:c26b2e31-d006-4d4d-9e56-45ca9391018d 2014-10-05 17:49:06 b.s.d.supervisor [INFO] Removing code for storm id production-topology-1-1412498729 2014-10-05 17:49:06 b.s.d.supervisor [INFO] Shutting down and clearing state for id 816a0fd6-7cdf-4312-a5ac-758f7f55ce46. Current supervisor time: 1412498946. State: :not-started, Heartbeat: nil 2014-10-05 17:49:06 b.s.d.supervisor [INFO] Shutting down 95111631-8f7f-46ed-a
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14159424#comment-14159424 ] ASF GitHub Bot commented on STORM-487: -- Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-57926268 @ChitturiPadma Thanks for sharing experience. I'll test on it. Btw, does storm.cmd runs normally with this scenario? > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14159418#comment-14159418 ] ASF GitHub Bot commented on STORM-487: -- Github user ChitturiPadma commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-57925935 I have tried this earlier. It runs the nimbus and supervisors without any issues and topology could be submitted without any issues. But when we try to kill the topology, supervisors would be terminated abruptly. This is happening because workers hold the topology jar file and internally clojure code first tries to remove the file and then kill the topology. Unfortunately this works in linux but windows doesnt allow to remove a file without closing the resources holding it. > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14159397#comment-14159397 ] ASF GitHub Bot commented on STORM-487: -- Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-57924831 @revans2 @ptgoetz Could you take a look, please? > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14157763#comment-14157763 ] ASF GitHub Bot commented on STORM-487: -- Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/280#issuecomment-57764337 In Windows, users should run command ```\python bin\storm blabla...```. > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14157760#comment-14157760 ] ASF GitHub Bot commented on STORM-487: -- GitHub user HeartSaVioR opened a pull request: https://github.com/apache/storm/pull/280 STORM-487 Let bin/storm compatible with Windows I modified bin/storm to be compatible with Windows. bin/storm script didn't take care of OS specific chars, so I've modified it. ps. IntelliJ said many lines in storm script violates PEP8. I wish to fix it, but it could hide original diff, so I will fix it by other PR after merge this in. You can merge this pull request into a Git repository by running: $ git pull https://github.com/HeartSaVioR/storm STORM-487 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/storm/pull/280.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #280 commit ed4861d83357dcb13cd32621fa18d1d03497a8a0 Author: Jungtaek Lim Date: 2014-10-03T07:18:40Z STORM-487 Let bin/storm compatible with Windows > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (STORM-487) Remove storm.cmd, no need to duplicate work python runs on windows too.
[ https://issues.apache.org/jira/browse/STORM-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14154044#comment-14154044 ] P. Taylor Goetz commented on STORM-487: --- I would love to see this resolved. Unfortunately, in my experience it's not as simple as it would seem. I can't remember exactly what the issues were (likely blocked from memory ;) ), but I was unable to get it to work even resorting to cygwin. The only thing that worked was the storm.cmd script originally provided by [~davidlao], hence its presence. If we can get storm.py to work on Windows without cygwin, I'd be all for it. > Remove storm.cmd, no need to duplicate work python runs on windows too. > --- > > Key: STORM-487 > URL: https://issues.apache.org/jira/browse/STORM-487 > Project: Apache Storm > Issue Type: Improvement >Reporter: Robert Joseph Evans > Labels: newbie > -- This message was sent by Atlassian JIRA (v6.3.4#6332)