sohami commented on a change in pull request #1677: DRILL-7068: Support memory 
adjustment framework for resource manageme…
URL: https://github.com/apache/drill/pull/1677#discussion_r263047886
 
 

 ##########
 File path: 
exec/java-exec/src/main/java/org/apache/drill/exec/physical/base/AbstractExchange.java
 ##########
 @@ -132,4 +132,20 @@ public int getOperatorType() {
   public ParallelizationDependency getParallelizationDependency() {
     return ParallelizationDependency.RECEIVER_DEPENDS_ON_SENDER;
   }
+
+  // Memory requirement of the sender for the given receivers and senders in a 
major fragment.
+  // Currently set to zero but later once batch sizing for Exchanges is 
completed it will calling
+  // apropriate function.
 
 Review comment:
   it will **call appropriate** Same for function below.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to