opensearch-project
Added from and to parameters to top N queries to specify time range
Description
Added from and to parameters to documentation for top N queries References PR: opensearch-project/query-insights#84 Issues PR: opensearch-project/query-insights#12
Checklist
- [x] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin. For more information on following Developer Certificate of Origin and signing off your commits, please check here.
Thank you for submitting your PR. The PR states are In progress (or Draft) -> Tech review -> Doc review -> Editorial review -> Merged.
Before you submit your PR for doc review, make sure the content is technically accurate. If you need help finding a tech reviewer, tag a maintainer.
When you're ready for doc review, tag the assignee of this PR. The doc reviewer may push edits to the PR directly or leave comments and editorial suggestions for you to address (let us know in a comment if you have a preference). The doc reviewer will arrange for an editorial review.
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
Thank you! Applying "Done but waiting to merge" label to put this in the 2.17 editorial queue starting 9/11. This PR will be merged before the release.
@kolchfa-aws Let's not merge this one in 2.17, since we still need some time to review the underlying PR: https://github.com/opensearch-project/query-insights/pull/84 and we might skip it for 2.17.
@ansjcy should we mark this for 2.18 if ew're not ready with the feature merge for 2.17 ?
@ansjcy Looks like the code is merged and backported to 2.x so this made it to 2.17. Removing the "on hold" label.
We are planing to ship this for 2.18 - as we merged the corresponding change into 2.x but didn't backport it into 2.17. Let's change the label to 2.18.