amansinha100 commented on a change in pull request #729: Drill 1328: Support 
table statistics for Parquet
URL: https://github.com/apache/drill/pull/729#discussion_r259152124
 
 

 ##########
 File path: 
exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java
 ##########
 @@ -911,4 +911,38 @@ public static String bootDefaultFor(String name) {
       new OptionDescription("Controls whether to return result set for CREATE 
TABLE / VIEW / FUNCTION, DROP TABLE / VIEW / FUNCTION, " +
           "SET, USE, REFRESH METADATA TABLE queries. If set to false affected 
rows count will be returned instead and result set will be null. " +
           "Affects JDBC connections only. Default is true. (Drill 1.15+)"));
+
+  /**
+   * Option whose value is a long value representing the number of bits 
required for computing ndv (using HLL).
+   * Controls the trade-off between accuracy and memory requirements. The 
number of bits correlates positively with accuracy
+   */
+  public static final String HLL_ACCURACY = "exec.statistics.ndv_accuracy";
+  public static final LongValidator HLL_ACCURACY_VALIDATOR = new 
PositiveLongValidator(HLL_ACCURACY, 30,
+      new OptionDescription("Controls trade-off between NDV statistic 
computation memory cost and accuracy"));
+
+  /**
+   * Option whose value is a long value representing the number of bits 
required for computing ndv (using HLL).
 
 Review comment:
   Comment needs to be updated. 

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


With regards,
Apache Git Services

Reply via email to