refactor(vortex-geo): generalize scalar function execution - #9215
refactor(vortex-geo): generalize scalar function execution#9215HarukiMoriarty wants to merge 1 commit into
Conversation
Merging this PR will degrade performance by 20.78%
Warning Please fix the performance issues or acknowledge them on CodSpeed. Performance Changes
Tip Investigate this regression by commenting Comparing Footnotes
|
5a42d1b to
751b43b
Compare
Signed-off-by: Nemo Yu <zyu379@wisc.edu>
751b43b to
09f16ff
Compare
Rationale for this change
Native geometry scalar functions need consistent handling for constant operands, pairwise columns, null propagation, and all-null batches. Keeping those responsibilities in each function makes new unary and binary kernels harder to implement correctly.
What changes are included in this PR?
What APIs are changed? Are there any user-facing changes?
This is an internal vortex-geo refactor. Existing scalar function semantics remain unchanged.