andygrove commented on PR #1683: URL: https://github.com/apache/datafusion-comet/pull/1683#issuecomment-2835058131
> Instead of passing a `ColumnDescriptor` from Iceberg to Comet, should we pass a String[] path, PrimitiveType.PrimitiveTypeName type, int maxRep, and int maxDef, so that we can construct the `ColumnDescriptor` on the Comet side? I did get farther, but now I run into the same problem when Iceberg calls the following method, because it needs to pass the Parquet PageReader class in: ``` public void setPageReader(PageReader pageReader) throws IOException ``` -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
