Skip to content

Conversation

@tlrx
Copy link
Member

@tlrx tlrx commented May 30, 2017

This commit adds support in ParsedMatrixStats for parsing the doc_count
field now it has been added in #24776.

This commit adds support in ParsedMatrixStats for parsing the doc_count
field.

Related to elastic#24776
Copy link
Contributor

@jimczi jimczi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tlrx tlrx merged commit eea010b into elastic:master May 30, 2017
@tlrx
Copy link
Member Author

tlrx commented May 30, 2017

Thanks @jpountz and @jimczi

@tlrx tlrx deleted the add-doc-count-to-parsed-matrix-stats branch May 30, 2017 08:19
tlrx added a commit that referenced this pull request May 30, 2017
This commit adds support in ParsedMatrixStats for parsing the doc_count
field.

Related to #24776
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request May 30, 2017
* 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)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants