Re: [PR] Add WITH ORDER example to blog post [datafusion-site]

2025-03-19 Thread via GitHub
alamb merged PR #59: URL: https://github.com/apache/datafusion-site/pull/59 -- 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...@datafusio

Re: [PR] Add WITH ORDER example to blog post [datafusion-site]

2025-03-18 Thread via GitHub
Omega359 commented on code in PR #59: URL: https://github.com/apache/datafusion-site/pull/59#discussion_r2002097220 ## content/blog/2025-03-11-ordering-analysis.md: ## @@ -291,6 +291,53 @@ Following third and fourth constraints for the simplified table, the succinct va `[time_

Re: [PR] Add WITH ORDER example to blog post [datafusion-site]

2025-03-18 Thread via GitHub
akurmustafa commented on PR #59: URL: https://github.com/apache/datafusion-site/pull/59#issuecomment-2734727273 > Thanks @akurmustafa > > I noticed some small format issues when reviewing this so I pushed [e10c17f](https://github.com/apache/datafusion-site/commit/e10c17f0f99b83d8e15cd

Re: [PR] Add WITH ORDER example to blog post [datafusion-site]

2025-03-18 Thread via GitHub
Omega359 commented on code in PR #59: URL: https://github.com/apache/datafusion-site/pull/59#discussion_r2002096546 ## content/blog/2025-03-11-ordering-analysis.md: ## @@ -291,6 +291,53 @@ Following third and fourth constraints for the simplified table, the succinct va `[time_

Re: [PR] Add WITH ORDER example to blog post [datafusion-site]

2025-03-18 Thread via GitHub
alamb commented on PR #59: URL: https://github.com/apache/datafusion-site/pull/59#issuecomment-2734731781 > Thanks @alamb, I was working on to add the example you gave ("DataFusion can find / use orderings based on query intermediates"). Should we add this to the document what do you think?

Re: [PR] Add WITH ORDER example to blog post [datafusion-site]

2025-03-18 Thread via GitHub
akurmustafa commented on PR #59: URL: https://github.com/apache/datafusion-site/pull/59#issuecomment-2734859868 With the [commit](https://github.com/apache/datafusion-site/pull/59/commits/85eea6a572f95972a155ee9926319112e7149ce8), I have added the @alamb's suggestion to the post. -- This

Re: [PR] Add WITH ORDER example to blog post [datafusion-site]

2025-03-18 Thread via GitHub
Omega359 commented on code in PR #59: URL: https://github.com/apache/datafusion-site/pull/59#discussion_r2002099018 ## content/images/ordering_analysis/query_window_plan.png: ## Review Comment: At the output of the window function the table has the ordering: -- This is

Re: [PR] Add WITH ORDER example to blog post [datafusion-site]

2025-03-18 Thread via GitHub
akurmustafa commented on PR #59: URL: https://github.com/apache/datafusion-site/pull/59#issuecomment-2734756045 > > Thanks @alamb, I was working on to add the example you gave ("DataFusion can find / use orderings based on query intermediates"). Should we add this to the document what do yo

Re: [PR] Add WITH ORDER example to blog post [datafusion-site]

2025-03-17 Thread via GitHub
alamb commented on code in PR #59: URL: https://github.com/apache/datafusion-site/pull/59#discussion_r1998872033 ## content/blog/2025-03-11-ordering-analysis.md: ## @@ -291,6 +291,31 @@ Following third and fourth constraints for the simplified table, the succinct va `[time_bin

[PR] Add WITH ORDER example to blog post [datafusion-site]

2025-03-16 Thread via GitHub
akurmustafa opened a new pull request, #59: URL: https://github.com/apache/datafusion-site/pull/59 @alamb and I added a blog post recently with the [PR](https://github.com/apache/datafusion-site/pull/58). We later thought that mentioning WITH ORDER API (which species orderings at the