You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Ensure Spark and Rapids plugin version of the exec match functionality.
They do, minor nit is we can have projectList be a seq(NamedExpression) instead of just seq(Expression) but that is minor and mostly for api one to one matching. Keeping this issue open to fix that. (EDIT: closing as this is benign)
Verify Config specific to the operator match.
It does.
Verify API is consistent and fully translated.
It is. There are some expression evaluation optimizations which we handle very differently so it should stay as is.
Port relevant tests.
Project exec is a simple exec that is used in many places and looking at the tests it is mostly for plan verification which I think is a benign change to our unit tests that seems unnecessary.
Is your feature request related to a problem? Please describe.
Ensure Spark and Rapids plugin version of the exec match functionality.
Verify Config specific to the operator match.
Verify API is consistent and fully translated.
Port relevant tests.
The text was updated successfully, but these errors were encountered: