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

Reply via email to