Skip to content

Conversation

@akuzm
Copy link
Member

@akuzm akuzm commented Jan 27, 2026

Transform it into an OR/AND bool operation and try to push down the suboperations individually.

We already have a bloom filter pusdown for x = any(array[....]), but this is more generic.

Transform it into an OR/AND bool operation and try to push down the
suboperations individually.
@akuzm akuzm requested a review from a team January 27, 2026 13:28
@akuzm akuzm mentioned this pull request Jan 27, 2026
8 tasks
@github-actions
Copy link

@melihmutlu, @fabriziomello: please review this pull request.

Powered by pull-review

@codecov
Copy link

codecov bot commented Jan 27, 2026

Codecov Report

❌ Patch coverage is 97.29730% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.60%. Comparing base (487e8c9) to head (9f342da).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
tsl/src/nodes/columnar_scan/qual_pushdown.c 97.29% 0 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9192      +/-   ##
==========================================
+ Coverage   82.47%   82.60%   +0.13%     
==========================================
  Files         245      245              
  Lines       48069    48117      +48     
  Branches    12253    12265      +12     
==========================================
+ Hits        39644    39749     +105     
- Misses       3578     3586       +8     
+ Partials     4847     4782      -65     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Co-authored-by: natalya-aksman <[email protected]>
Signed-off-by: Alexander Kuzmenkov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants