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

Reply via email to