You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2015/10/14 23:15:05 UTC
[jira] [Commented] (STORM-720) Storm.cmd should return ERRORLEVEL
before exiting main block
[ https://issues.apache.org/jira/browse/STORM-720?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14957780#comment-14957780 ]
ASF GitHub Bot commented on STORM-720:
--------------------------------------
Github user knusbaum commented on a diff in the pull request:
https://github.com/apache/storm/pull/476#discussion_r42054199
--- Diff: bin/storm.cmd ---
@@ -90,6 +90,7 @@
)
set path=%PATH%;%STORM_BIN_DIR%;%STORM_SBIN_DIR%
call start /b "%storm-command%" "%JAVA%" %JAVA_HEAP_MAX% %STORM_OPTS% %STORM_LOG_FILE% %CLASS% %storm-command-arguments%
+ exit /b %ERRORLEVEL%
goto :eof
--- End diff --
We can get rid of this `goto :eof` now, right?
> Storm.cmd should return ERRORLEVEL before exiting main block
> ------------------------------------------------------------
>
> Key: STORM-720
> URL: https://issues.apache.org/jira/browse/STORM-720
> Project: Apache Storm
> Issue Type: Bug
> Components: storm-core
> Affects Versions: 0.9.3, 0.9.4
> Reporter: Ravi Tandon
> Assignee: Ravi Tandon
> Priority: Trivial
> Fix For: 0.9.3, 0.9.4
>
> Attachments: storm_cmd_exitcode_rev0.patch
>
>
> This JIRA is for a very small PR that I will post soon (attached patch)
> Issue: The Storm.cmd does not exit with an ErrorLevel.
> Impact: Any automation via windows banks on return or exit code from the program to determine success.
> When can this occur: Passing wrong arguments (like wrong class name or bad topology builder code) to java will result in error and any automation program will check on the exit code.
> FYI @ [~harsha_ch] [~shanyu]
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)