[GitHub] metron pull request #1252: METRON-1855: Make unified enrichment topology the...

[mmiklavc]

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

    https://github.com/apache/metron/pull/1252#discussion_r230864542
  
    --- Diff: metron-platform/metron-enrichment/README.md ---
    @@ -76,6 +62,19 @@ intel bolt, the configurations will be taken from the 
respective join bolt
     parallelism.  When proper ambari support for this is added, we will add
     its own property.
     
    +### Split-Join Enrichment Topology
    +
    +The now-deprecated split/join topology is also available and performs 
enrichments in parallel.
    +This poses some issues in terms of ease of tuning and reasoning about 
performance.
    +
    +![Architecture](enrichment_arch.png)
    +
    +#### Using It
    +
    +In order to use the older, deprecated topology, you will need to
    +* Edit `$METRON_HOME/bin/start_enrichment_topology.sh` and adjust it to 
use `remote-splitjoin.yaml` instead of `remote-unified.yaml`
    --- End diff --
    
    This actually wasn't net-new, it's the way we currently expose the choice. 
I would prefer to keep it as-is and manual because the desire is NOT to have 
customers use the deprecated topology.


---