xuchuanyin commented on a change in pull request #3063: [CARBONDATA-3242] Move Range_Column into the table level properties
URL:
https://github.com/apache/carbondata/pull/3063#discussion_r247351978
##########
File path: core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
##########
@@ -1176,6 +1177,15 @@ private CarbonCommonConstants() {
*/
public static final int SORT_SIZE_MIN_VAL = 1000;
+ /**
+ * For Range_Column, it will use SCALE_FACTOR to control the size of each partition.
+ * When SCALE_FACTOR is about the compression ratio, each task will generate one CarbonData file.
+ * And the size of the file is about TABLE_BLOCKSIZE of this table.
Review comment:
Difficult to understand these two sentences...
Please consider to optimize them for better understanding.
----------------------------------------------------------------
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:
[hidden email]
With regards,
Apache Git Services