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

    https://github.com/apache/flink/pull/4911#discussion_r154965118
  
    --- Diff: 
flink-core/src/main/java/org/apache/flink/api/common/operators/ResourceSpec.java
 ---
    @@ -37,11 +44,24 @@
      *     <li>Direct Memory Size</li>
      *     <li>Native Memory Size</li>
      *     <li>State Size</li>
    + *     <li>Extended resources</li>
      * </ol>
      */
     @Internal
     public class ResourceSpec implements Serializable {
     
    +   public enum ResourceAggregateType {
    +           /**
    +            * Denotes keeping the sum of the values with same name when 
merging two resource specs for operator chaining
    +            */
    +           AGGREGATE_TYPE_SUM,
    +
    +           /**
    +            * Denotes keeping the max of the values with same name when 
merging two resource specs for operator chaining
    +            */
    +           AGGREGATE_TYPE_MAX
    +   }
    --- End diff --
    
    Let's move this enum to `Resource`


---

Reply via email to