Re: [PR] Fix: Preserve sorting for the COPY TO plan [datafusion]

2025-07-17 Thread via GitHub
alamb commented on PR #16785: URL: https://github.com/apache/datafusion/pull/16785#issuecomment-3085556000 Thanks again @bert-beyondloops -- 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 spec

Re: [PR] Fix: Preserve sorting for the COPY TO plan [datafusion]

2025-07-17 Thread via GitHub
alamb merged PR #16785: URL: https://github.com/apache/datafusion/pull/16785 -- 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...@datafusi

Re: [PR] Fix: Preserve sorting for the COPY TO plan [datafusion]

2025-07-16 Thread via GitHub
alamb closed pull request #16785: Fix: Preserve sorting for the COPY TO plan URL: https://github.com/apache/datafusion/pull/16785 -- 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.

Re: [PR] Fix: Preserve sorting for the COPY TO plan [datafusion]

2025-07-16 Thread via GitHub
alamb commented on PR #16785: URL: https://github.com/apache/datafusion/pull/16785#issuecomment-3078438440 close/reopen to retrigger CI -- 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 specifi

Re: [PR] Fix: Preserve sorting for the COPY TO plan [datafusion]

2025-07-16 Thread via GitHub
bert-beyondloops commented on PR #16785: URL: https://github.com/apache/datafusion/pull/16785#issuecomment-3077507961 Thanks @alamb for the quick 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

Re: [PR] Fix: Preserve sorting for the COPY TO plan [datafusion]

2025-07-15 Thread via GitHub
alamb commented on code in PR #16785: URL: https://github.com/apache/datafusion/pull/16785#discussion_r2208580080 ## datafusion/core/tests/dataframe/mod.rs: ## @@ -6193,3 +6194,59 @@ async fn test_copy_schema() -> Result<()> { assert_logical_expr_schema_eq_physical_expr_sch

[PR] Fix: Preserve sorting for the COPY TO plan [datafusion]

2025-07-15 Thread via GitHub
bert-beyondloops opened a new pull request, #16785: URL: https://github.com/apache/datafusion/pull/16785 ## Which issue does this PR close? will close issue #16784 ## Rationale for this change The INSERT INTO already takes account of the ordering requirements during plan