siddharthteotia commented on a change in pull request #6869:
URL: https://github.com/apache/incubator-pinot/pull/6869#discussion_r630481681
##########
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/recommender/rules/io/params/RecommenderConstants.java
##########
@@ -89,6 +90,13 @@
public static final int[] DEFAULT_NUM_HOSTS = {3, 4, 5, 6, 7, 8, 9, 10,
11, 12};
}
+ public static class SegmentSizeRule {
+ public static final int DEFAULT_NUM_SEGMENTS = 1;
+ public static final int DEFAULT_DESIRED_SEGMENT_SIZE_MB = 500;
+ public static final int NOT_PROVIDED = -1;
+ public static final double INDEX_OVERHEAD_RATIO_FOR_SEGMENT_SIZE = 1.2;
Review comment:
This isn't configurable right ?
--
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
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
