[STORM-2663] Backport STORM-2558 and deprecate storm.cmd on 1.x-branch

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.1.0, 1.1.1, 1.2.0
• Fix Version/s: 1.2.0, 1.1.2
• Component/s: storm-core
• Assignee: Stig Rohde Døssing
• Reporter: Stig Rohde Døssing
• Created: 29/Jul/17 15:22
• Updated: 31/Jul/17 03:00
• Resolved: 31/Jul/17 03:00

Description

The storm.cmd script has been removed from master, but it's still very difficult to work with on 1.x-branch, due to several deficiencies compared to storm.py. It is missing certain commands (pacemaker at least) and not setting some Java properties correctly (-DdaemonName and the Java classpath are set incorrectly). Rather than spending time fixing these issues, I'd like to deprecate storm.cmd in future 1.x releases and add the Powershell script as an alternative for Windows users.

Issue Links

• is related to: STORM-2558 The Windows command line storm.cmd script is outdated and doesn't work
• is related to: STORM-2641 storm.py and storm.ps1 don't handle errors correctly on Windows
• relates to: STORM-2662 Fix for STORM-2659 is incomplete, some commands still don't have daemonName set correctly
• links to: GitHub Pull Request #2246