alamb commented on issue #13065:
URL: https://github.com/apache/datafusion/issues/13065#issuecomment-2445424416

   > My vision we should correctly implement Eq for Schema or provide a method 
that checks everything but nullability and metadata.
   
   I think it is somewhat debatable if we should/shouldn't be checking that the 
nullability and metadata matches. I actually think having the invariant that 
the corresponding physical plan created for a LogicalPlan should have the exact 
same schema seems quite reasonable 🤔 


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to