search

apache / datafusion-comet

Apache DataFusion Comet Spark Accelerator
https://datafusion.apache.org/comet
Apache License 2.0
823 stars 163 forks source link

chore: Refactor UnaryExpr and MathExpr in protobuf #1056

Closed andygrove closed 2 weeks ago

andygrove commented 2 weeks ago

Which issue does this PR close?

N/A

Rationale for this change

This PR follows on from https://github.com/apache/datafusion-comet/pull/1053 which refactored BinaryExpr to reduce code duplication. This PR does the same for MathExpr and UnaryExpr.

What changes are included in this PR?

Just refactoring to reduce code duplication. There are no functional changes.

How are these changes tested?

Existing tests.