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

    https://github.com/apache/storm/pull/2419#discussion_r150950610
  
    --- Diff: 
storm-client/src/jvm/org/apache/storm/trident/topology/TridentTopologyBuilder.java
 ---
    @@ -51,8 +51,8 @@
     
     // based on transactional topologies
     public class TridentTopologyBuilder {
    -    Map<GlobalStreamId, String> _batchIds = new HashMap();
    -    Map<String, TransactionalSpoutComponent> _spouts = new HashMap();
    +    Map<GlobalStreamId, String> batchIds = new HashMap();
    --- End diff --
    
    Nit: Raw types here and in the following lines


---

Reply via email to