rdettai edited a comment on pull request #965:
URL: https://github.com/apache/arrow-datafusion/pull/965#issuecomment-911505188
TODO:
- [x] move the `statistics()` function to the `ExecutionPlan` trait and
provide naive implementations
- [x] re-implement the aggregate_statistics CBO
- [x] re-implement the hash_build_probe_order CBO
- [ ] explain/analyze using default stat ok?
- [x] union using accumulator ok?
- made scalar min and max public instead
- [x] test: hash_build_probe_order integration (sql.rs)
- [x] test: hash_build_probe_order unit test
- [x] test: union stats
- [x] test: stat for record batch helper
- [x] test: projection stats
- [x] test: cross join stats
- [x] test: limit stats
- [ ] test: window agg stat
- [x] test: ballista merge partition stats (shuffle_reader)
- [x] should we really use async?
- decided to make it sync and consider the plan as immutable
--
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]
