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

order by primary keys if no order_by is provided #282

Merged
merged 1 commit into from Mar 12, 2014

Conversation

@glyphobet
Copy link
Contributor

@glyphobet glyphobet commented Feb 5, 2014

We were seeing test failures in our project because the ordering of search results is not necessarily deterministic if no order_by is specified. This makes API responses deterministic; the same request always returns the same ordering of results.

jfinkels added a commit that referenced this issue Mar 12, 2014
order by primary keys if no order_by is provided
@jfinkels jfinkels merged commit df64d95 into jfinkels:master Mar 12, 2014
1 check passed
Loading
@glyphobet glyphobet deleted the ordered-search branch Mar 15, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants