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?
---