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

ASF GitHub Bot commented on DRILL-4134:
---------------------------------------

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

    https://github.com/apache/drill/pull/283#discussion_r46225385
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/ops/FragmentContext.java ---
    @@ -314,11 +285,16 @@ public BufferAllocator getAllocator() {
         return allocator;
       }
     
    -  public BufferAllocator getNewChildAllocator(final long 
initialReservation,
    -                                              final long 
maximumReservation,
    -                                              final boolean 
applyFragmentLimit) throws OutOfMemoryException {
    -    return allocator.getChildAllocator(new AsLimitConsumer(), 
initialReservation, maximumReservation,
    -        applyFragmentLimit);
    +  public BufferAllocator getNewChildAllocator(final String operatorName,
    +      final int operatorId,
    +      final long initialReservation,
    +      final long maximumReservation,
    +      final boolean applyFragmentLimit) throws OutOfMemoryException {
    +    return allocator.newChildAllocator(
    +        "op:" + QueryIdHelper.getFragmentId(fragment.getHandle()) + ":" + 
operatorId + ":" + operatorName,
    +        initialReservation,
    +        maximumReservation
    +        );
    --- End diff --
    
    applyFragmentLimit is not used?


> Incorporate remaining patches from DRILL-1942 Allocator refactor
> ----------------------------------------------------------------
>
>                 Key: DRILL-4134
>                 URL: https://issues.apache.org/jira/browse/DRILL-4134
>             Project: Apache Drill
>          Issue Type: Sub-task
>          Components: Execution - Flow
>            Reporter: Jacques Nadeau
>            Assignee: Jacques Nadeau
>             Fix For: 1.4.0
>
>




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

Reply via email to