[GitHub] metron issue #789: METRON-1233: Remove description of Global configuration f...

[mmiklavc]

Github user mmiklavc commented on the issue:

https://github.com/apache/metron/pull/789
  
@DimDroll @ottobackwards - All of our topologies pull in the global config. 
One such more recent example is that we now provide an option for specifying 
batching details for enrichment:
```
enrichment.writer.batchSize
enrichment.writer.batchTimeout
```

The reason for this is that we don't have a global-only-per-topology type 
of configuration, with the exception parsers because of how they can be 
deployed independently. If you think there's some better clarification that 
could be made, I'm open to it. But I think the link between parsers, 
enrichment, indexing, and the global configs should be maintained as it is 
relevant.




---

[GitHub] metron issue #789: METRON-1233: Remove description of Global configuration f...

[ottobackwards]

Github user ottobackwards commented on the issue:

https://github.com/apache/metron/pull/789
  
bump.  Let's get a response on this @mmiklavc , @JonZeolla , @nickwallen 


---

[GitHub] metron issue #789: METRON-1233: Remove description of Global configuration f...

[DimDroll]

Github user DimDroll commented on the issue:

https://github.com/apache/metron/pull/789
  
@mmiklavc , I still don't see anything helpful specified about enrichment 
in here:

https://metron.apache.org/current-book/metron-platform/metron-common/index.html
I guess we can reference enrichment configuration in global configuration, 
but I still don't see sense of mentioning Global configuration in Enrichment 
section without anything relevant to enrichment in Global configuration.


---

[GitHub] metron issue #789: METRON-1233: Remove description of Global configuration f...

[ottobackwards]

Github user ottobackwards commented on the issue:

https://github.com/apache/metron/pull/789
  
Still needs conflict resolution, still needs relevancy check


---