weizhengte commented on code in PR #12987: URL: https://github.com/apache/doris/pull/12987#discussion_r984304410
########## 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 feel this is not necessary, we'd better try to keep it consistent with other databases.`stat` feels more like the meaning of status, `stats` means statistical information, And `ColumnStats` is used to describe the statistics of the original table column. For the problem of naming the collection of nereids, the columnStats of the collection type uses `columnsStats`, Instead of `columnStatss`, does this also work? Of course, I think these can be discussed further. -- 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