You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "alamb (via GitHub)" <gi...@apache.org> on 2023/06/22 10:16:33 UTC

[GitHub] [arrow-datafusion] alamb commented on issue #6741: RFC: Rename OptimizerConfig to OptimizerContext

alamb commented on issue #6741:
URL: https://github.com/apache/arrow-datafusion/issues/6741#issuecomment-1602383119

   Makes sense to me too
   
   The one thing I worry about is reusing the `OptimizerContext`
   
   >  existing OptimizerContext which is a concrete class should be renamed to OptimizerState.
   
   Which could make an upgrade quite confusing as the errors won't be about a type not being present, but instead will be about it being the wrong class. Maybe we could find a different name
   


-- 
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-unsubscribe@arrow.apache.org

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