Github user nickwallen commented on the issue:

    https://github.com/apache/metron/pull/740
  
    
    I am re-reading through everyone's comments just to make sure I haven't 
misjudged all the feedback that I have received so far (thanks to all for 
feedback).  
    
    > you could also make %define nicks_new_global_field := my_complex_field 
That makes somewhat more sense since stellar assignment is := and property 
assignment is =
    
    I think I misjudged this one @cestella .  I like this for a number of 
reasons.
    * This is still very simple for the user.  
    * This is more consistent with the existing assignment construct.
    * This makes it far easier to build complex types for a global %define; its 
easier to build a Stellar map, than a JSON string where the quotes need escaped 
in Stellar.
    
    So the simple case becomes this.
    ```
    %define bootstrap.servers: = node1:6667
    ```
    
    And the more complex case becomes this.
    ```
    value := { "foo": "bar" }
    %define bootstrap.servers := value
    ```



---

Reply via email to