BlakeOrth commented on PR #17232: URL: https://github.com/apache/datafusion/pull/17232#issuecomment-3226017592
@alamb @nuno-faria I believe I touched all the asks from the previous round of feedback with the latest commit. I referenced a recent PR by @nuno-faria where the parquet metadata cache option was added to help guide me through getting a new config option pushed into all the correct locations. That PR had changes in the various `proto` directories, but I looked and it didn't seem like this option belongs to a set of options that are serialized, so I didn't think there were any changes necessary there. Please let me know if I missed anything! -- 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
