metron-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From nickwallen <...@git.apache.org>
Subject [GitHub] metron issue #879: METRON-1378: Create a summarizer
Date Tue, 16 Jan 2018 15:23:43 GMT
Github user nickwallen commented on the issue:

    https://github.com/apache/metron/pull/879
  
    +0  I'm sure what's here is solid, but I have not reviewed it myself.  I just want to
clear the way for this to get merged.
    
    I don't necessarily like the usability of this approach, but I think it is a good first
step.  And merging this doesn't preclude providing alternative approaches or enhancing this
approach for better usability later.  
    
    I do love the use case that was added that drove the need for this functionality.  So
good stuff @cestella!


---

Mime
View raw message