houqp commented on issue #1228: URL: https://github.com/apache/arrow-datafusion/issues/1228#issuecomment-966036390
Good point @xudong963 , totally forgot about object safety :( I don't have a strong opinion on this and I am cool with us going either way. I do agree with you that adding the `TableScanPlan` struct makes things more flexible in the long run. -- 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]
