-
Notifications
You must be signed in to change notification settings - Fork 91
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
support date_nanos type when select time field for creating monitor #954
Conversation
can we check why e2e are failing? |
OpenSearch cluster failed to start will fixed by opensearch-project/alerting#1558, after that we can retry the CI |
@riysaxen-amzn @AWSHurneyt can you help to re-run the failed CI? |
@Hailong-am I can't seem to rerun the CI. Could you submit an empty commit to get them to run? |
Signed-off-by: Hailong Cui <ihailong@amazon.com>
@AWSHurneyt rebase from main and all CIs are passed @riysaxen-amzn would you mind to help to review? |
@Hailong-am Does this need any backporting? |
please help to add backport 2.x label, thanks |
Signed-off-by: Hailong Cui <ihailong@amazon.com> (cherry picked from commit ce4742b) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
(cherry picked from commit ce4742b) Signed-off-by: Hailong Cui <ihailong@amazon.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Description
Add date_nanos support for time field selection
test date_nanos field with range query which used in define the query condition in creating monitor, it works as expected without exception.
Issues Resolved
#927
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.