berkaysynnada commented on PR #13293:
URL: https://github.com/apache/datafusion/pull/13293#issuecomment-2467866670
I prefer going with
```
pub enum Estimate {
Range { bounds: Interval, value: ScalarValue },
/// Nothing is known about the value
#[default]
Absent,
}
```
as mentioned
[there](https://github.com/apache/datafusion/issues/8078#issuecomment-2466779170).
We will be both making us of interval arithmetics, and eliminating the need
for separate bound and estimation statistics. It does not necessitate to select
which kind of stats (range or estimate) you keep.
The only challenge is we need to guard the internal values, and provide the
all API's someone can require. It should be never in an inconsistent state
(like estimate value is out of the bounds).
--
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: github-unsubscr...@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org
