Skip to content

Commit

Permalink
doc: clarify nullability information source for empty literals
Browse files Browse the repository at this point in the history
  • Loading branch information
vbarua committed Apr 3, 2024
1 parent e564cf0 commit 08ad96f
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions proto/substrait/algebra.proto
Original file line number Diff line number Diff line change
Expand Up @@ -825,9 +825,11 @@ message Expression {
UserDefined user_defined = 33;
}

// whether the literal type should be treated as a nullable type. Applies to
// all members of union other than the Typed null (which should directly
// declare nullability).
// Whether the literal_type above should be treated as a nullable type.
// Applies to all members of the literal_type oneof EXCEPT:
// * Type null (must be nullable by definition)
// * Type.List empty_list (use Type.List::nullability)
// * Type.Map empty_map (use Type.Map::nullability)
bool nullable = 50;

// optionally points to a type_variation_anchor defined in this plan.
Expand Down

0 comments on commit 08ad96f

Please sign in to comment.