Dandandan commented on a change in pull request #620:
URL: https://github.com/apache/arrow-datafusion/pull/620#discussion_r659339127
##########
File path: datafusion/src/datasource/datasource.rs
##########
@@ -108,6 +108,11 @@ pub trait TableProvider: Sync + Send {
/// Statistics should be optional because not all data sources can provide
statistics.
fn statistics(&self) -> Statistics;
+ /// Returns whether statistics provided are exact values or estimates
Review comment:
That's a similar thought process I had.
Maybe at some point it would also be nice to tell what parts of the
statistics are exact (e.g. number of rows) and what estimated (such as distinct
count).
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
