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

    https://github.com/apache/metron/pull/1099#discussion_r201361910
  
    --- Diff: 
metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/topology/ParserTopologyBuilder.java
 ---
    @@ -91,14 +101,14 @@ public Config getTopologyConfig() {
        */
       public static ParserTopology build(String zookeeperUrl,
                                           Optional<String> brokerUrl,
    -                                      String sensorType,
    -                                      ValueSupplier<Integer> 
spoutParallelismSupplier,
    -                                      ValueSupplier<Integer> 
spoutNumTasksSupplier,
    +                                      List<String> sensorTypes,
    +                                      ValueSupplier<List> 
spoutParallelismSupplier,
    --- End diff --
    
    Ah, I see.  man, I miss C++ templates sometimes.  I retract the nit; don't 
worry about it.


---

Reply via email to