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

    https://github.com/apache/storm/pull/2563#discussion_r169180931
  
    --- Diff: 
storm-client/src/jvm/org/apache/storm/trident/topology/TridentTopologyBuilder.java
 ---
    @@ -779,30 +769,20 @@ public BoltDeclarer addConfigurations(Map<String, 
Object> conf) {
                 return this;
             }
     
    +        /**
    +         * return the current component configuration.
    +         *
    +         * @return the current configuration.
    +         */
             @Override
    -        public BoltDeclarer addResources(Map<String, Double> resources) {
    -            if (resources != null) {
    -                Map<String, Double> currentResources = (Map<String, 
Double>) component.componentConf.computeIfAbsent(
    -                    Config.TOPOLOGY_COMPONENT_RESOURCES_MAP, (k) -> new 
HashMap<>());
    -                currentResources.putAll(resources);
    -            }
    -            return this;
    +        public Map<String, Object> getComponentConfiguration() {
    +            return component.componentConf;
             }
     
    -        @Override
    +       @Override
    --- End diff --
    
    nit: broken indent


---

Reply via email to