NotFoundOvO commented on PR #23110: URL: https://github.com/apache/datafusion/pull/23110#issuecomment-4850777564
Hi @alamb, you're right — I checked and PR #23143 has already removed the `RowIndex` struct. The other items in this PR (`raw_data`, `check_arrow_schema_type_compatible`, `try_new_with_having`) may still be candidates, but since this branch now has conflicts and was partially superseded, I'll close this PR. If those items are still needed I can open smaller individual PRs. Thanks for the review! -- 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]
