substrait-io / substrait

A cross platform way to express data transformation, relational algebra, standardized record expression and plans.
https://substrait.io
Apache License 2.0
1.19k stars 155 forks source link

fix: correct format for nullable interval_day parameters #687

Closed Blizzara closed 2 months ago

Blizzara commented 2 months ago

Fixes a bug in functions_datetime.yaml that makes it un-parseable by substrait-java, introduced in #679. The ? for nullability must come before the parameters: https://substrait.io/types/type_parsing/#type-syntax-parsing

jacques-n commented 2 months ago

Really begs completing #686 and running the parser in substrait core.

Thanks @Blizzara !