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

Fixing query to field status to use integer instead of timestamp string #10684

Merged
merged 2 commits into from
Mar 3, 2017

Conversation

simianhacker
Copy link
Member

Changes the query to field_stats to use integers instead of ISO8601 strings. Fixes #10683

Copy link
Contributor

@thomasneirynck thomasneirynck left a comment

Choose a reason for hiding this comment

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

change looks good to me

@thomasneirynck
Copy link
Contributor

@alexfrancoeur do you want to give it a 2nd glance?

@alexfrancoeur
Copy link

@thomasneirynck @simianhacker we're golden - thanks for the quick turnaround! Any chance we can merge?

screen shot 2017-03-03 at 10 45 41 am

@simianhacker simianhacker merged commit 1808e88 into elastic:master Mar 3, 2017
elastic-jasper added a commit that referenced this pull request Mar 3, 2017
Backports PR #10684

**Commit 1:**
Fixing query to field status to use integer instead of timestamp string

* Original sha: 1a71e66
* Authored by Chris Cowan <[email protected]> on 2017-03-03T13:02:15Z

**Commit 2:**
Adding format; fixing tests

* Original sha: 9c80b98
* Authored by Chris Cowan <[email protected]> on 2017-03-03T13:13:24Z
simianhacker pushed a commit that referenced this pull request Mar 3, 2017
…ng (#10688)

Backports PR #10684

**Commit 1:**
Fixing query to field status to use integer instead of timestamp string

* Original sha: 1a71e66
* Authored by Chris Cowan <[email protected]> on 2017-03-03T13:02:15Z

**Commit 2:**
Adding format; fixing tests

* Original sha: 9c80b98
* Authored by Chris Cowan <[email protected]> on 2017-03-03T13:13:24Z
@simianhacker simianhacker deleted the fix-10683 branch March 16, 2017 21:33
coverslide pushed a commit to coverslide/kibana that referenced this pull request Mar 17, 2017
…ng (elastic#10684)

* Fixing query to field status to use integer instead of timestamp string

* Adding format; fixing tests
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.

4 participants