bipinprasad opened a new pull request #3302:
URL: https://github.com/apache/storm/pull/3302


   ## What is the purpose of the change
   
   Rebalance command completes successfully when --executor option has invalid 
component name. This gives
   the impression that command was successful. Change this behavior to throw an 
exception. In addition, fix the help
   text for --executor options to show correct format.
   
   ## How was the change tested
   
   Run storm and rebalance using various correct and incorrect formats like:
     bin/storm rebalance --help   # to show corrected help text
     bin/storm rebalance --executor spout:3 -w 1 word-count ## fail because of 
spout:3
     bin/storm rebalance --executor spout=3 -w 1 word-count ## verify correct 
rebalance in Storm UI with spout=3
     bin/storm rebalance -e spout=4 -w 1 word-count  ## verify correct 
rebalance in Storm UI with spout=4
     


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to