[GitHub] drill pull request #1181: DRILL-6284: Add operator metrics for batch sizing ...

[paul-rogers]

Github user paul-rogers commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1181#discussion_r178445230
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/join/MergeJoinBatch.java
 ---
    @@ -109,12 +110,37 @@
       private static final String LEFT_INPUT = "LEFT INPUT";
       private static final String RIGHT_INPUT = "RIGHT INPUT";
     
    -  private class MergeJoinMemoryManager extends 
AbstractRecordBatchMemoryManager {
    +  private static final int numInputs = 2;
    +  private static final int LEFT_INDEX = 0;
    +  private static final int RIGHT_INDEX = 1;
    +
    +  public enum Metric implements MetricDef {
    +    LEFT_INPUT_BATCH_COUNT,
    +    LEFT_AVG_INPUT_BATCH_BYTES,
    +    LEFT_AVG_INPUT_ROW_BYTES,
    +    LEFT_TOTAL_INPUT_RECORDS,
    --- End diff --
    
    `LEFT_INPUT_RECORD_COUNT`?


---