Github user vrozov commented on the issue:

    https://github.com/apache/drill/pull/1237
  
    IMO, it is better not to report memory usage at all compared to reporting a 
wrong number. In case incoming batches are accumulated in a queue, they should 
be reported as owned by a receiver. Taking ownership just before passing a 
batch to the next operator does not sound right to me.
    
    I don't think it is necessary to create new fragment child allocator. 
Receiver allocator should be used instead of fragment allocator when an 
incoming batch is placed into a queue.


---

Reply via email to