substrait-io / substrait-java

Apache License 2.0
74 stars 70 forks source link

fix: handle custom extensions through expressions #161

Closed vbarua closed 11 months ago

vbarua commented 11 months ago

ProtoExpressionConverter was creating a new default ProtoRelConverter to translate rels contained in expressions. This caused problems when those Rels contain extensions, because the default ProtoRelConverter instance did not know how to handle them.

These changes pass a ProtoRelConverter to the ProtoExpressionConverter, which it uses to translate expression.