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.
---