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 range queries with date range based on current time in percolator queries. #24666

Merged
merged 1 commit into from May 16, 2017

Conversation

Projects
None yet
3 participants
@martijnvg
Copy link
Member

commented May 15, 2017

Range queries with now based date ranges were previously not allowed,
but since #23921 these queries were allowed. This change should really
fix range queries with now based date ranges.

@martijnvg martijnvg changed the title Fix range queries with date range based on current time. Fix range queries with date range based on current time in percolator queries. May 15, 2017

@jimczi

jimczi approved these changes May 15, 2017

Copy link
Member

left a comment

LGTM

@clintongormley clintongormley added v5.4.2 and removed v5.4.1 labels May 15, 2017

percolator: Fix range queries with date range based on current time.
Range queries with now based date ranges were previously not allowed,
but since #23921 these queries were allowed. This change should really
fix range queries with now based date ranges.

@martijnvg martijnvg force-pushed the martijnvg:percolator_range_with_now_fix branch to f6e19dc May 16, 2017

@martijnvg martijnvg merged commit f6e19dc into elastic:master May 16, 2017

1 of 2 checks passed

elasticsearch-ci Build started sha1 is merged.
Details
CLA Commit author is a member of Elasticsearch
Details

@clintongormley clintongormley added v5.4.1 and removed v5.4.2 labels May 22, 2017

@clintongormley clintongormley added v6.0.0-beta1 and removed v6.0.0 labels Jul 25, 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.