[ 
https://issues.apache.org/jira/browse/FLINK-2237?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15140786#comment-15140786
 ] 

ASF GitHub Bot commented on FLINK-2237:
---------------------------------------

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

    https://github.com/apache/flink/pull/1517#discussion_r52459053
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/io/disk/RandomAccessInputView.java
 ---
    @@ -45,9 +45,12 @@ public RandomAccessInputView(ArrayList<MemorySegment> 
segments, int segmentSize)
        {
                this(segments, segmentSize, segmentSize);
        }
    -   
    +
    +   /**
    +    * Warning: setReadPosition has to be called before reading.
    +    */
        public RandomAccessInputView(ArrayList<MemorySegment> segments, int 
segmentSize, int limitInLastSegment) {
    --- End diff --
    
    Can you add a new constructor instead of modifying the behavior of this one?


> Add hash-based Aggregation
> --------------------------
>
>                 Key: FLINK-2237
>                 URL: https://issues.apache.org/jira/browse/FLINK-2237
>             Project: Flink
>          Issue Type: New Feature
>            Reporter: Rafiullah Momand
>            Assignee: Gabor Gevay
>            Priority: Minor
>
> Aggregation functions at the moment are implemented in a sort-based way.
> How can we implement hash based Aggregation for Flink?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to