Github user justinleet commented on the issue:

    https://github.com/apache/metron/pull/1249
  
    @ottobackwards I was sick the past couple days, so sorry about the delay in 
responding.
    
    Hopefully, what we're saying is, for this PR, refactor this to
    ```
    └── metron-parsing
        ├── metron-parsers
        ├── metron-parsers-common
        └── metron-parsing-storm
    ```
    
    Where metron-parsers-common contains the various utils, plus the building 
block parsers (e.g. Grok).  Whenever another platform gets added, it would go 
in "metron-parsing-<platform>" and they can create "metron-parsing-common" if 
needed (because right now it would be empty).
    
    Sound like a plan?


---

Reply via email to