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

Fix script field sort returning Double.MAX_VALUE for all documents #24942

Merged
merged 1 commit into from May 30, 2017

Conversation

Projects
None yet
3 participants
@jimczi
Copy link
Member

commented May 29, 2017

This change fixes the script field sort when the returned type is a number.

Closes #24940

Fix script field sort returning Double.MAX_VALUE for all documents
This change fixes the script field sort when the returned type is a number.

Closes #24940

@jimczi jimczi merged commit 628dabd into elastic:master May 30, 2017

2 checks passed

CLA Commit author is a member of Elasticsearch
Details
elasticsearch-ci Build finished.
Details

@jimczi jimczi deleted the jimczi:script_field_sort branch May 30, 2017

jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request May 30, 2017

Merge branch 'master' into primary-promotion-transition
* master:
  Fix typo in comment in ReplicationOperation.java
  Prevent Index & Delete request primaryTerm getter/setter, setShardId setter
  Drop name from TokenizerFactory (elastic#24869)
  Correctly set doc_count when MovAvg "predicts" values on existing buckets (elastic#24892)
  Handle primary failure handling replica response
  Add missing word to terms-query.asciidoc (elastic#24960)
  Correct some spelling in match-phrase-prefix docs (elastic#24956)
  testConcurrentWriteViewsAndSnapshot shouldn't flush concurrently
  [TEST] Fix FieldSortIT failures
  Add doc_count to ParsedMatrixStats (elastic#24952)
  Add document count to Matrix Stats aggregation response (elastic#24776)
  Fix script field sort returning Double.MAX_VALUE for all documents (elastic#24942)

@clintongormley clintongormley added v6.0.0-alpha2 and removed v6.0.0 labels Jun 6, 2017

@jimczi jimczi restored the jimczi:script_field_sort branch Sep 28, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.