[ 
https://issues.apache.org/jira/browse/DRILL-8465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17792114#comment-17792114
 ] 

ASF GitHub Bot commented on DRILL-8465:
---------------------------------------

pjfanning opened a new pull request, #2853:
URL: https://github.com/apache/drill/pull/2853

   ## Description
   
   https://issues.apache.org/jira/browse/DRILL-8465
   
   I'm not happy with the class check here. I don't know to what extent that we 
need to support subclasses that a user might make of Iceberg classes. If we 
need to support subclasses of Iceberg classes, it might be better to support a 
config option that allows users to extend the allow list for classes in this 
deserialization code.
   
   In this PR, the allow list is:
   * java primitives
   * java/javax classes
   * org.apache.iceberg classes
   * org.apache.drill classes
   * arrays of the types listed above
   * messiest bit is the support for any subclass of iceberg ScanTask class




> check data input when loading iceberg data
> ------------------------------------------
>
>                 Key: DRILL-8465
>                 URL: https://issues.apache.org/jira/browse/DRILL-8465
>             Project: Apache Drill
>          Issue Type: Improvement
>          Components: Storage - Iceberg
>            Reporter: PJ Fanning
>            Priority: Major
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to