Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[10.x] Followup Fix sql server ordering for distinct columns #47766

Closed
wants to merge 5 commits into from

Conversation

joelharkes
Copy link
Contributor

@joelharkes joelharkes commented Jul 17, 2023

Followup #47763

trying to fix test cases for distinct + limit and paging

@joelharkes
Copy link
Contributor Author

I don't understand why the query builder still fails in the paginate function? 🤷

@joelharkes
Copy link
Contributor Author

it gives me:

 select distinct top 15 [users].* from [users] inner join [posts] on [posts].[user_id] = [users].[id] order by (SELECT 0)

but i'd expect me to give it:

 select distinct top 15 [users].* from [users] inner join [posts] on [posts].[user_id] = [users].[id] order by (SELECT [users].*)

side note: sql server cannot distinct on single column

@driesvints
Copy link
Member

@joelharkes looks like this PR needs some more work on its tests like you noticed. For this, we suggest you try a support channel in an attempt to figure out why they fail. It could very well be that there's no real solution here. If you have a PR with passing tests, feel free to send that in 👍

@driesvints driesvints closed this Sep 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants