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