rdblue commented on a change in pull request #2952:
URL: https://github.com/apache/iceberg/pull/2952#discussion_r693533713
##########
File path: api/src/main/java/org/apache/iceberg/types/TypeUtil.java
##########
@@ -42,6 +42,45 @@
private TypeUtil() {
}
+ /**
+ * Project extracts particular fields from a schema. Unlike
+ * {@link TypeUtil#select(Schema, Set)}, project will pick out only
+ * the fields enumerated, this means no sub fields will be selected
+ * unless they are explicitly requested.
Review comment:
You might add "... unless they are explicitly requested. Structs that
are explicitly projected are empty unless sub-fields are explicitly projected.
Maps and lists cannot be explicitly selected in fieldIds."
--
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]
