Github user nickwallen commented on the issue:

    https://github.com/apache/metron/pull/740
  
    > Can't we turn everything passed in into json and use the json patch stuff 
to "build out' the configuration? Why not have the globals as a 
map<string,object> all the time?
    
    It still is always a map<string, object>, that has not changed. When I said 
"How about we just restrict %define to strings only for now.", that just means 
that a `%define` drops into the map<string, object>, values of strings without 
trying to interpret the value and create a complex type.
    
    The main itch that I'm trying to scratch here (which is maybe the 80% 
case?) is to just set a simple property value.
    ```
    %define bootstrap.servers=node1:6667
    KAFKA_GET("indexing", 10)
    ```
    
    If we treat `%define` arguments as JSON (so it behaves just like the `-z`) 
then it would have to look something like this, which is way more quotes and 
parens than I'd like to deal with.
    ```
    %define { "bootstrap.servers" : "node1:6667" }
    KAFKA_GET("indexing", 10)
    ```


---

Reply via email to