refactor(pojos)!: avoid using raw proto enums in POJO layer

substrait-io / substrait-java

Apache License 2.0

75 stars 72 forks source link

refactor(pojos)!: avoid using raw proto enums in POJO layer #164

Closed vbarua closed 1 year ago

vbarua commented 1 year ago

BREAKING CHANGE: various public functions that took the AggregateFunction.AggregationInvocation proto now take the POJO equivalent Expression.AggregationInvocation.

vbarua commented 1 year ago

AggregateFunction.AggregationInvocation was the only protobuf enum that didn't have a POJO equivalent. I added one for consistency while I was looking into additional support for window functions.