You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@storm.apache.org by danny0405 <gi...@git.apache.org> on 2018/06/06 04:29:33 UTC

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

Github user danny0405 commented on the issue:

    https://github.com/apache/storm/pull/2703
  
    Sorry i don't think put the validation at Client side is a good idea, cause many kind of clients may interact with Master through thrift RPC, we must keep all the clients sync on this validation rule if we moved it to client side.
    
    Now the client kind i know:
    
    - java script 
    - java
    - python
    
    Even if we all modified and sync the rule between all the client, we still can not prevent some private modification to break it.


---