opensearch-project / opensearch-project/sql
[ENHANCEMENT] Adding filter for bucketNullable=false by hint strategy
@qianheng-aws is already working on this.
Since Sep 29, 2025.
- Dominant language
- Java
- Stars
- 176
- Forks
- 229
- Avg merge
- 2d 21h
- Merged PRs (30d)
- 43
Description
Is your feature request related to a problem?
This is a follow up to #3550, which introduce a new configuration of bucketNullable.
That PR implements bucketNullable=false by adding a IS_NOT_NULL FILTER in logical plan. However this new filter operator is redundant since we already has hint(BUCKET_NULLABLE=false) in LogicalAggregate to implies null bucket is not included, and it will also prevent aggregate push down in some cases.
What solution would you like?
Add that filter in planning process by hint strategy instead of directly add it in logical plan.
What alternatives have you considered?
A clear and concise description of any alternative solutions or features you've considered.
Do you have any additional context?
Add any other context or screenshots about the feature request here.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.