feat(expr): add bound expressions - #9125
Conversation
Signed-off-by: Joe Isaacs <joe.isaacs@live.co.uk>
Codecov Report❌ Patch coverage is
☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Merging this PR will degrade performance by 48.47%
Warning Please fix the performance issues or acknowledge them on CodSpeed. Performance Changes
Tip Investigate this regression by commenting Comparing Footnotes
|
Summary
Scopeand a type-checkedBoundExpressiontree whose nodes retain their inferred dtypes.Expression::return_dtypeas a compatibility path backed by expression binding.Why
Consumers that know the authoritative root dtype should be able to bind once and retain type information instead of recursively recomputing it throughout analysis and execution.
Impact
This is PR 1 of 2 in the stack. It adds the bound-expression representation and compatibility API without changing scan or layout interfaces.
Stack
Validation
cargo check -p vortex-arraycargo +nightly fmt --all -- --checkgit diff --check