substrait-io / substrait-java

Apache License 2.0
72 stars 70 forks source link

feat: add support for empty list literals #227

Closed patientstreetlight closed 4 months ago

patientstreetlight commented 4 months ago

The substrait proto definition provides a representation of empty list literals which is separate from the representation of non-empty list literals:

message Literal {
  oneof literal_type {
    ...
    List list = 30;
    Type.List empty_list = 31;
    ...
  }
}

This change gives substrait-java a representation for empty list literals, along with conversions to/from protobuf and to/from Calcite in Isthmus.

BREAKING CHANGE:

CLAassistant commented 4 months ago

CLA assistant check
All committers have signed the CLA.