Kikyou1997 commented on code in PR #12987: URL: https://github.com/apache/doris/pull/12987#discussion_r985164385
########## fe/fe-core/src/main/java/org/apache/doris/statistics/ColumnStat.java: ########## @@ -54,7 +51,7 @@ * For example: * "@ndv = 10" means that the number distinct values is 10 in the whole table. */ -public class ColumnStats { +public class ColumnStat { Review Comment: @morrySnow ########## fe/fe-core/src/main/java/org/apache/doris/statistics/ColumnStat.java: ########## @@ -54,7 +51,7 @@ * For example: * "@ndv = 10" means that the number distinct values is 10 in the whole table. */ -public class ColumnStats { +public class ColumnStat { Review Comment: I will mark all the fields of ColumnStat and TabletStat as `final` in another PR, since this PR is a little huge now. I think this discussion could be ended. If you have any other question, you could open a issue or propose it on the weekly discussion. -- 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. To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org