alamb commented on PR #11587: URL: https://github.com/apache/datafusion/pull/11587#issuecomment-2246042385
I think what we should do here is: 1. File a follow on ticket based on @2010YOUY01 's comment https://github.com/apache/datafusion/pull/11587#discussion_r1686678665 2. Merge this PR (it targets the `string-view2` branch, not main) Per @ozankabak 's comment https://github.com/apache/datafusion/pull/11610#issuecomment-2244524858 there appears to be another PR https://github.com/synnada-ai/datafusion-upstream/pull/27 that will affect `CoelseceBatchesExec` and thus we may have a merge conflict to resolve when bringing `string-view2` back into main However I think that is manageable and we should be ok -- 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
