Skip to content

Conversation

@DrakeLin
Copy link
Collaborator

@DrakeLin DrakeLin commented Jan 21, 2026

🥞 Stacked PR

Use this link to review incremental changes.


What changes are proposed in this pull request?

This PR adds an Expression::coalesce helper method to simplify creating COALESCE expressions, and updates all existing call sites to use the new helper.

How was this change tested?

Existing tests

@codecov
Copy link

codecov bot commented Jan 21, 2026

Codecov Report

❌ Patch coverage is 98.75000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 84.16%. Comparing base (435b94e) to head (287577f).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...src/engine/arrow_expression/evaluate_expression.rs 94.44% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1648   +/-   ##
=======================================
  Coverage   84.15%   84.16%           
=======================================
  Files         123      123           
  Lines       34180    34252   +72     
  Branches    34180    34252   +72     
=======================================
+ Hits        28764    28827   +63     
+ Misses       4021     4016    -5     
- Partials     1395     1409   +14     

☔ 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.

@github-actions github-actions bot added the breaking-change Change that require a major version bump label Jan 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking-change Change that require a major version bump

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant