Github user nickwallen commented on a diff in the pull request:

    https://github.com/apache/metron/pull/984#discussion_r180166554
  
    --- Diff: 
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/configuration/metron-enrichment-env.xml
 ---
    @@ -129,70 +145,156 @@
       <property>
         <name>enrichment_topology_max_spout_pending</name>
         <description>Spout Max Pending Tuples for the Enrichment 
Topology</description>
    -    <value/>
    -    <display-name>Enrichment Max Pending</display-name>
    +    <value>500</value>
    --- End diff --
    
    But this is definitely not necessary for the task at hand; METRON-1499.  
    
    If this is contentious, I can certainly pull this change out of this PR.  I 
could then submit a separate PR where we can debate this point.  I don't want 
to hang up this PR for something unrelated.  
    
    I'm good with either way.


---

Reply via email to