adriangb commented on issue #16800:
URL: https://github.com/apache/datafusion/issues/16800#issuecomment-3079371233

   Thank you for chiming in!
   
   > We'll be able to start making the migration with the DF 49.0 release?
   
   Yes that's the plan. I'm trying to figure out the way to make the migration 
as incremental and painless as possible. We'll also have to be doing this 
migration. The current plan as outlined in 
https://github.com/apache/datafusion/pull/16791 is that by default if you don't 
provide anything you use the new system. If you have a custom SchemaAdapter it 
will be used for everything. You can start migrating by implementing a 
PhysicalExprRewriter that has 1:1 pairity with your SchemaAdapter 
implementation and passing that in _as well as_ your SchemaAdapter, then the 
SchemaAdapter will be used for projections and the PhysicalExprRewriter will be 
used for predicate pushdown.
   
   Once we implement projection pushdown without any changes to your code you'd 
be fully migrated to the new system and can just remove the SchemaAdapter.


-- 
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: github-unsubscr...@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to