You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by Yuliya Feldman <yu...@yahoo.com> on 2015/01/13 02:34:05 UTC

Review Request 29835: DRILL-1926 - Fix backp ressure logic

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29835/
-----------------------------------------------------------

Review request for drill, Aditya Kishore, Jacques Nadeau, and Steven Phillips.


Bugs: DRILL-1926
    https://issues.apache.org/jira/browse/DRILL-1926


Repository: drill-git


Description
-------

Fix Back Pressure logic where it was kicking in only when number of elements in the queue was == to softLimit.
Also change to release back presure gradually and not in a single shot


Diffs
-----

  exec/java-exec/src/main/java/org/apache/drill/exec/work/batch/ResponseSenderQueue.java 5a9316ebac83e39c6a66b3d4835bb2cea303908b 
  exec/java-exec/src/main/java/org/apache/drill/exec/work/batch/UnlimitedRawBatchBuffer.java 623a719d8ff54554f7e77bcda7b50de66b48c766 
  exec/java-exec/src/test/java/org/apache/drill/exec/work/batch/TestUnlimitedBatchBuffer.java PRE-CREATION 

Diff: https://reviews.apache.org/r/29835/diff/


Testing
-------

all suites


Thanks,

Yuliya Feldman