dramaticlly commented on code in PR #17626:
URL: https://github.com/apache/iceberg/pull/17626#discussion_r3777651936
##########
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/SparkSchemaUtil.java:
##########
@@ -249,30 +232,6 @@ public static Schema prune(Schema schema, StructType
requestedType) {
.fields());
}
- /**
- * Prune columns from a {@link Schema} using a {@link StructType Spark type}
projection.
- *
- * <p>This requires that the Spark type is a projection of the Schema.
Nullability and types must
- * match.
- *
- * <p>The filters list of {@link Expression} is used to ensure that columns
referenced by filters
- * are projected.
- *
- * @param schema a Schema
- * @param requestedType a projection of the Spark representation of the
Schema
- * @param filters a list of filters
- * @return a Schema corresponding to the Spark projection
- * @throws IllegalArgumentException if the Spark type does not match the
Schema
- */
- public static Schema prune(Schema schema, StructType requestedType,
List<Expression> filters) {
Review Comment:
I reverted the change for spark 3.5 for now.
--
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]
