You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2022/12/12 11:56:30 UTC

[GitHub] [flink-table-store] zjureel commented on a diff in pull request #426: [FLINK-30323] Support table statistics in table store

zjureel commented on code in PR #426:
URL: https://github.com/apache/flink-table-store/pull/426#discussion_r1045733557


##########
flink-table-store-core/src/main/java/org/apache/flink/table/store/table/source/Split.java:
##########
@@ -23,5 +23,5 @@
 /** An input split for reading. */
 public interface Split extends Serializable {
 
-    long rowCount();

Review Comment:
   Agree, I think the column stats should be optimized instead of reading from file meta directly and we can discuss about it later



-- 
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-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org