adriangb commented on PR #20202: URL: https://github.com/apache/datafusion/pull/20202#issuecomment-3872184302
@kosiew could you help me understand your long term vision here? Are we going to fold `cast_column(...)` into `cast(...)` or something else? Also at least the PR description is outdated: > Simplified and clarified SpillPoolWriter example by removing unnecessary concurrent task spawning from the doc snippet. I don't see this change anywhere (and if it is here IMO is should be it's own PR, it seems like a valuable but drive by improvement unrelated to the rest of these changes). -- 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]
