danielcweeks commented on code in PR #12476: URL: https://github.com/apache/iceberg/pull/12476#discussion_r1992245698
########## core/src/main/java/org/apache/iceberg/InternalData.java: ########## @@ -163,6 +180,11 @@ public interface ReadBuilder { /** Set a custom class for in-memory objects at the given field ID. */ ReadBuilder setCustomType(int fieldId, Class<? extends StructLike> structClass); + /** Set the classloader used for custom types. */ + default ReadBuilder classLoader(ClassLoader classLoader) { + throw new UnsupportedOperationException("Classloader not supported"); Review Comment: I couldn't find why this was necessary since the builder itself should have the same classloader. I was thinking there must be some usage elsewhere but I don't see anything. I'll remove this. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org