waynexia commented on PR #4607: URL: https://github.com/apache/arrow-datafusion/pull/4607#issuecomment-1371126226
>Makes SchemaProvider::table async. This change makes sense to me. It's a good solution to only change one interface, I like it! The only thing left is asynchronous register things, but on the one hand it's less important than getting and deregistering, and another hand there are still ways to workaround (like update-on-getting). So I'm +0.5 for providing this change as-is. Thanks for making this @tustvold 🥳 -- 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]
