rdblue commented on a change in pull request #2877:
URL: https://github.com/apache/iceberg/pull/2877#discussion_r681323979
##########
File path: api/src/main/java/org/apache/iceberg/util/StructProjection.java
##########
@@ -83,7 +83,7 @@ private StructProjection(StructType structType, StructType
projection) {
break;
case MAP:
case LIST:
- throw new IllegalArgumentException(String.format("Cannot project
list or map field: %s", projectedField));
+ // TODO Figure this out
Review comment:
What about allowing the projection if the the fields are primitives or
if the entire struct is projected? That would cover the cases that are
currently supported and avoid introducing a new pruning bug to replace the one
you're fixing (where nested structs don't match the requested struct schema).
--
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]
