bvolpato commented on code in PR #25944:
URL: https://github.com/apache/datafusion/pull/25944#discussion_r4240281246
##########
docs/source/library-user-guide/upgrading/56.0.0.md:
##########
@@ -788,3 +788,38 @@ let plan = provider
.await?
.into_inner();
```
+
+### Generated protobuf `EmptyRelationNode` gained a `schema` field
+
+The generated public protobuf `EmptyRelationNode` struct now carries an
+optional `schema` so logical plans reduced to an `EmptyRelation` (for example
+by `LIMIT 0` or `WHERE false`) keep their columns across serialization.
+
+**Who is affected:**
+
+- Users constructing generated `EmptyRelationNode` values with an exhaustive
Review Comment:
nonblocking: could we also mention that `EmptyRelationNode` loses `Copy`,
`Eq`, and `Hash`? Adding `schema: None` fixes struct literals, but callers
relying on those bounds or using it as a `HashMap` key still need to migrate.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
