xudong963 commented on issue #1228:
URL: 
https://github.com/apache/arrow-datafusion/issues/1228#issuecomment-960541366


   > Something that is more of a hack but might be quicker to do would be to 
use a `NewType` kind of thing
   
   Good idea!
   
   
   
   > I think extracting `LogicalPlan` into structs would be fine, though it 
will be a very non trivial change (both for DataFusion and all users)
   
   Long-term thinking, maybe we can profit from extracting `LogicalPlan` into 
structs. So if all of you agree, I can do it before doing subquery features. cc 
@houqp


-- 
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]


Reply via email to