[GitHub] storm pull request #2713: [STORM-3094] : Added topology name validation at c...

2018-06-13 Thread ManoharVanam
GitHub user ManoharVanam opened a pull request: https://github.com/apache/storm/pull/2713 [STORM-3094] : Added topology name validation at client side Current Behavior : Execute topology with invalid topology name is throwing exception after uploading the jar. Improvement

[GitHub] storm issue #2703: [STORM-3094] : Added topology name validation at client s...

2018-06-11 Thread ManoharVanam
Github user ManoharVanam commented on the issue: https://github.com/apache/storm/pull/2703 @HeartSaVioR done. ---

[GitHub] storm issue #2703: [STORM-3094] : Added topology name validation at client s...

2018-06-07 Thread ManoharVanam
Github user ManoharVanam commented on the issue: https://github.com/apache/storm/pull/2703 Yes , here we are handling two things, 1. Fail fast without uploading the jar 2. Giving clear message to the user at the client side. ---

[GitHub] storm pull request #2703: [STORM-3094] : Added topology name validation at c...

2018-06-05 Thread ManoharVanam
GitHub user ManoharVanam opened a pull request: https://github.com/apache/storm/pull/2703 [STORM-3094] : Added topology name validation at client side Current Behavior : Execute topology with invalid topology name is throwing exception after uploading the jar. Improvement