gparai commented on a change in pull request #1715: DRILL-7117: Support creation of equi-depth histogram for selected dat… URL: https://github.com/apache/drill/pull/1715#discussion_r269752646
########## File path: exec/java-exec/src/main/java/org/apache/drill/exec/planner/common/DrillStatsTable.java ########## @@ -62,10 +62,15 @@ public enum STATS_VERSION {V0, V1}; // The current version public static final STATS_VERSION CURRENT_VERSION = STATS_VERSION.V1; + // 10 histogram buckets (TODO: can make this configurable later) + public static final int NUM_HISTOGRAM_BUCKETS = 10; Review comment: Maybe we can look at other implementations to figure out a good default value for histogram buckets. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to 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