Github user nickwallen commented on the issue:

    https://github.com/apache/metron/pull/970
  
    Thanks @justinleet for consuming my rant. :)
    
    I agree with most of what you're saying.  I'm perfectly happy iterating 
improvements on this.  And totally willing to help.  One disagreement...
    
    > For the specific calculating metascores case, and probably some of the 
other stuff, it only seems useful to break that sort of thing into it's own 
class if we intend on making it pluggable.
    
    I disagree here.  Making things pluggable, even if we now currently only 
have one type of plug, is still beneficial.  It is going to help make it more 
testable and shrink the square footage of that class.



---

Reply via email to