Github user ilooner commented on a diff in the pull request:

    https://github.com/apache/drill/pull/970#discussion_r145764765
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet2/DrillParquetReader.java
 ---
    @@ -331,21 +320,21 @@ public int next() {
         return count;
       }
     
    -  private int getPercentFilled() {
    -    int filled = 0;
    -    for (final ValueVector v : primitiveVectors) {
    -      filled = Math.max(filled, v.getAccessor().getValueCount() * 100 / 
v.getValueCapacity());
    -      if (v instanceof VariableWidthVector) {
    -        filled = Math.max(filled, ((VariableWidthVector) 
v).getCurrentSizeInBytes() * 100 / ((VariableWidthVector) v).getByteCapacity());
    -      }
    -      // TODO - need to re-enable this
    -//      if (v instanceof RepeatedFixedWidthVector) {
    -//        filled = Math.max(filled, ((RepeatedFixedWidthVector) 
v).getAccessor().getGroupCount() * 100)
    +//  private int getPercentFilled() {
    --- End diff --
    
    Delete?


---

Reply via email to