Skip to content

Add outer nullability to Union#8797

Merged
connortsui20 merged 0 commit into
agent/union-scalar-nullabilityfrom
agent/union-dtype-nullability
Jul 16, 2026
Merged

Add outer nullability to Union#8797
connortsui20 merged 0 commit into
agent/union-scalar-nullabilityfrom
agent/union-dtype-nullability

Conversation

@connortsui20

@connortsui20 connortsui20 commented Jul 16, 2026

Copy link
Copy Markdown
Member

Tracking issue: #8769

Splits the breaking Union dtype change out of #8791.

Following #8769, we are going to track both inner and outer / top-level nullability in union types.

Changes

  • changes DType::Union to carry independent outer nullability
  • keeps outer nullability separate from the nullability of each variant
  • updates dtype coercion, serialization, display, scalar construction, and downstream matches
  • distinguishes an outer null Union from a present Union whose selected child is null

This changes the serialized Serde representation of Union dtypes and will require the compatibility override to be reviewed explicitly.

This was referenced Jul 16, 2026
@connortsui20 connortsui20 added the changelog/break A breaking API change label Jul 16, 2026
@connortsui20
connortsui20 force-pushed the agent/union-dtype-nullability branch from e035b08 to c5f26f5 Compare July 16, 2026 17:36
@connortsui20
connortsui20 merged commit c5f26f5 into agent/union-scalar-nullability Jul 16, 2026
@connortsui20
connortsui20 force-pushed the agent/union-scalar-nullability branch from af088ad to 3a22a55 Compare July 16, 2026 17:36
@connortsui20
connortsui20 deleted the agent/union-dtype-nullability branch July 16, 2026 17:36
@connortsui20

Copy link
Copy Markdown
Member Author

Superseded by #8798. During the stack reorder, GitHub marked this PR merged into its former stacked base; it did not merge the nullability change into develop.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/break A breaking API change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant