yahoNanJing commented on PR #2906: URL: https://github.com/apache/arrow-datafusion/pull/2906#issuecomment-1185557326
> Thanks @yahoNanJing -- I understand the need for this PR and would like to get it in to unblock you if needed > > The only thing that this PR needs, in my opinion, is a test (of the ObjectStoreDetector logic) > > @tustvold and I had some suggestions on how to improve the code but I think they could be done as a follow on PR as well Actually, this feature is only ballista's interests currently. And I'll add a test in the ballista repo. However, due to the dependency issue among datafusion, ballista and datafusion-objectstore-hdfs. It's necessary for us to merge this PR first. 😢 -- 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]
