findepi commented on PR #16573: URL: https://github.com/apache/datafusion/pull/16573#issuecomment-3031159421
> So it is still my opinion that copying fields from `ConfigProperties` into` ExecutionProps` one at a time will over time result in more and more "hair" (aka we'll have some subset of the fields and it won't be clear why some are included and some are not) That was definitely my intention -- expose only those fields or state attributes that are relevant. Keep the API surface as small as only sufficient. (I need blah, so i pass-through blah is a trivial change to make by anyone who needs blah in the future.) If that's controversial, or problematic, then sure we can expose all the config options like in https://github.com/apache/datafusion/pull/16661 -- 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
