ohad-definity commented on PR #59706: URL: https://github.com/apache/spark/pull/59706#issuecomment-6578676501
@sunchao thanks for the thorough re-review. Both optional doc clarifications are in f286bc11: the internal flag now says ORC scans are not covered yet, and the `hasBlockingPushdown` comment names its generator-path caller. I'd keep the two planning improvements (equal schemas in `keepMapKeys`, repeated suffix analysis on V1 no-op paths) as a follow-up. The [fork Build](https://github.com/ohad-definity/spark/actions/runs/41436071851) on f286bc11 is green. If this looks good to you, could you approve or dismiss the earlier change request? -- 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]
