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

Review request for drill.


Repository: drill-git


Description
-------

* Inserted a unique Metric class into each RootExec class to allow different 
* Removed the OperatorStats extension SenderStats (from BaseRootExec as well as 
project)
  * These metrics were only being reported by PartitionSender
  * SenderStats does not write waitTime to the query profile
* report max and min # records metrics directly from PartitionSender
* report # receivers from PartitionSender and BroadcastSender
* report # of bytes sent for all different types of senders (including Screen)
* rename N_SENDERS to N_RECEIVERS


Diffs
-----

  exec/java-exec/src/main/java/org/apache/drill/exec/ops/OperatorStats.java 
dcb73c8 
  exec/java-exec/src/main/java/org/apache/drill/exec/ops/SenderStats.java 
c766632 
  
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/BaseRootExec.java
 452052b 
  
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/ScreenCreator.java
 9ad85af 
  
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/SingleSenderCreator.java
 1b63112 
  
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/broadcastsender/BroadcastSenderRootExec.java
 a70cd50 
  
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/materialize/QueryWritableBatch.java
 c219cce 
  
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/PartitionSenderRootExec.java
 c4844d5 
  
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/PartitionSenderStats.java
 de5967f 
  
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/PartitionerTemplate.java
 6a26d30 
  
exec/java-exec/src/main/java/org/apache/drill/exec/record/FragmentWritableBatch.java
 ef7b5f2 

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


Testing
-------


Thanks,

Cliff Buchanan

Reply via email to