Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEA] Add Hybrid scan tests for the large list of expressions that are listed as supported #12136

Closed
revans2 opened this issue Feb 13, 2025 · 1 comment
Labels
duplicate This issue or pull request already exists feature request New feature or request

Comments

@revans2
Copy link
Collaborator

revans2 commented Feb 13, 2025

Is your feature request related to a problem? Please describe.
We recently ran into an issue where timestamps didn't work properly with hybrid scan predicate push down.

#12112

When reviewing the code I noticed that there was a very large list of expressions that are supported as a part of hybrid scan predicate push down.

#12112 (review)

But we have almost no coverage for any of these.

At a minimum we need at least one test for each of these, and ideally we have more to validate that they are doing the right thing.

@revans2 revans2 added ? - Needs Triage Need team to review and classify feature request New feature or request labels Feb 13, 2025
@revans2
Copy link
Collaborator Author

revans2 commented Feb 13, 2025

Sorry this is a dupe of #12124

@revans2 revans2 closed this as not planned Won't fix, can't repro, duplicate, stale Feb 13, 2025
@sameerz sameerz added duplicate This issue or pull request already exists and removed ? - Needs Triage Need team to review and classify labels Feb 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists feature request New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants