Zouxxyy commented on PR #10714: URL: https://github.com/apache/incubator-gluten/pull/10714#issuecomment-3299606067
> Could we extend it to support V2? This fix iceberg, but other data lake may also need it. I had considered adding it here before, but the prerequisite is that we need a unified validation before applying `injectFakeRowAdaptor`. Currently, we are using `doValidateInternal` in `ColumnarV2TableWriteExec` to do the validation in offload rule, which hasn't been executed at this point yet. I can extract a common validation logic, or would you have any other suggestions. -- 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]
