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

API Change: search sort #113

Closed
clintongormley opened this issue Apr 4, 2010 · 3 comments
Closed

API Change: search sort #113

clintongormley opened this issue Apr 4, 2010 · 3 comments

Comments

@clintongormley
Copy link

Instead of specifying sort order like this:
"sort" : [
"name",
{
"last_modified" : {
"reverse" : 1
}
}
],

What about like this?
"sort" : [
"name",
{"last_modified" : "desc"}
],

or

"sort" : [
  {"name" : "asc"},
  {"last_modified" : "desc"}
],

When you sort on score, reverse: true implies asc, but for everything else, reverse: true implies desc. So why not make it unambiguous?

@kimchy
Copy link
Member

kimchy commented Apr 9, 2010

Going to support it as well as the old way

@kimchy
Copy link
Member

kimchy commented Apr 9, 2010

implemented

@clintongormley
Copy link
Author

Tested++

dadoonet pushed a commit that referenced this issue Jun 5, 2015
rmuir pushed a commit to rmuir/elasticsearch that referenced this issue Nov 8, 2015
Some exceptions might not be serializable. It would be safer not to wrap them in a `MapperParsingException` but just create the `MapperParsingException`.

Related to elastic#113.
(cherry picked from commit e58878c)
(cherry picked from commit a673185)
cbuescher pushed a commit to cbuescher/elasticsearch that referenced this issue Oct 2, 2023
With this commit we add the machine information for the new nightly
environment and also clarify that the definitive reference is not this
list but rather our SSH configuration.

Relates elastic#113
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants