davidradl commented on code in PR #25950:
URL: https://github.com/apache/flink/pull/25950#discussion_r1910216285
##########
flink-table/flink-table-common/src/main/java/org/apache/flink/table/plan/stats/ColumnStats.java:
##########
@@ -118,40 +81,10 @@ public Integer getMaxLen() {
return maxLen;
}
- /**
- * Deprecated because Number type max/min is not well supported comparable
type, e.g. {@link
- * java.util.Date}, {@link java.sql.Timestamp}.
- *
- * <p>Returns null if this instance is constructed by {@link
ColumnStats.Builder}.
- */
- @Deprecated
- public Number getMaxValue() {
- return maxValue;
- }
-
- /**
Review Comment:
It looks like you have removed the javadoc associated with the getMax method
--
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: issues-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
