added support for searching multiple rollup indices with same mapping #440
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com>
Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com>
Codecov Report
@@ Coverage Diff @@
## main #440 +/- ##
============================================
- Coverage 75.84% 75.80% -0.05%
- Complexity 2465 2467 +2
============================================
Files 313 313
Lines 14426 14411 -15
Branches 2230 2228 -2
============================================
- Hits 10942 10924 -18
- Misses 2240 2245 +5
+ Partials 1244 1242 -2
Help us with your feedback. Take ten seconds to tell us how you rate us. |
Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com>
…b; Added picking rollup job deterministic Signed-off-by: petar.dzepina <petar.dzepina@dev22.rs>
Signed-off-by: petar.dzepina <petar.dzepina@dev22.rs>
downsrob
approved these changes
Aug 5, 2022
Angie-Zhang
approved these changes
Aug 5, 2022
opensearch-trigger-bot bot
pushed a commit
that referenced
this pull request
Aug 5, 2022
…#440) * added support for searching multiple rollup indices with same mapping Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com> * fixed failing rollupInterceptorIT test Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com> * reverted old error messages Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com> * reverted checking for matching jobs on whole set instead of job by job; Added picking rollup job deterministic Signed-off-by: petar.dzepina <petar.dzepina@dev22.rs> * fixed sorting Signed-off-by: petar.dzepina <petar.dzepina@dev22.rs> Co-authored-by: petar.dzepina <petar.dzepina@dev22.rs> (cherry picked from commit b8a77d4)
downsrob
pushed a commit
that referenced
this pull request
Aug 5, 2022
…#440) (#447) * added support for searching multiple rollup indices with same mapping Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com> * fixed failing rollupInterceptorIT test Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com> * reverted old error messages Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com> * reverted checking for matching jobs on whole set instead of job by job; Added picking rollup job deterministic Signed-off-by: petar.dzepina <petar.dzepina@dev22.rs> * fixed sorting Signed-off-by: petar.dzepina <petar.dzepina@dev22.rs> Co-authored-by: petar.dzepina <petar.dzepina@dev22.rs> (cherry picked from commit b8a77d4) Co-authored-by: Petar Dzepina <petar.dzepina@gmail.com>
6 tasks
petardz
added a commit
to petardz/index-management
that referenced
this pull request
Aug 9, 2022
…opensearch-project#440) * added support for searching multiple rollup indices with same mapping Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com> * fixed failing rollupInterceptorIT test Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com> * reverted old error messages Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com> * reverted checking for matching jobs on whole set instead of job by job; Added picking rollup job deterministic Signed-off-by: petar.dzepina <petar.dzepina@dev22.rs> * fixed sorting Signed-off-by: petar.dzepina <petar.dzepina@dev22.rs> Co-authored-by: petar.dzepina <petar.dzepina@dev22.rs>
Angie-Zhang
pushed a commit
to Angie-Zhang/index-management
that referenced
this pull request
Sep 12, 2022
…opensearch-project#440) (opensearch-project#447) * added support for searching multiple rollup indices with same mapping Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com> * fixed failing rollupInterceptorIT test Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com> * reverted old error messages Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com> * reverted checking for matching jobs on whole set instead of job by job; Added picking rollup job deterministic Signed-off-by: petar.dzepina <petar.dzepina@dev22.rs> * fixed sorting Signed-off-by: petar.dzepina <petar.dzepina@dev22.rs> Co-authored-by: petar.dzepina <petar.dzepina@dev22.rs> (cherry picked from commit b8a77d4) Co-authored-by: Petar Dzepina <petar.dzepina@gmail.com> Signed-off-by: Angie Zhang <langelzh@amazon.com>
wuychn
pushed a commit
to ochprince/index-management
that referenced
this pull request
Mar 16, 2023
…opensearch-project#440) (opensearch-project#447) * added support for searching multiple rollup indices with same mapping Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com> * fixed failing rollupInterceptorIT test Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com> * reverted old error messages Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com> * reverted checking for matching jobs on whole set instead of job by job; Added picking rollup job deterministic Signed-off-by: petar.dzepina <petar.dzepina@dev22.rs> * fixed sorting Signed-off-by: petar.dzepina <petar.dzepina@dev22.rs> Co-authored-by: petar.dzepina <petar.dzepina@dev22.rs> (cherry picked from commit b8a77d4) Co-authored-by: Petar Dzepina <petar.dzepina@gmail.com>
ronnaksaxena
pushed a commit
to ronnaksaxena/index-management
that referenced
this pull request
Jul 19, 2023
…opensearch-project#440) (opensearch-project#447) * added support for searching multiple rollup indices with same mapping Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com> * fixed failing rollupInterceptorIT test Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com> * reverted old error messages Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com> * reverted checking for matching jobs on whole set instead of job by job; Added picking rollup job deterministic Signed-off-by: petar.dzepina <petar.dzepina@dev22.rs> * fixed sorting Signed-off-by: petar.dzepina <petar.dzepina@dev22.rs> Co-authored-by: petar.dzepina <petar.dzepina@dev22.rs> (cherry picked from commit b8a77d4) Co-authored-by: Petar Dzepina <petar.dzepina@gmail.com> Signed-off-by: Ronnak Saxena <ronsax@amazon.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Petar Dzepina petar.dzepina@gmail.com
Issue #, if available: 321
Description of changes:
Allowed searching multiple rollup indices with matching field mappings from query
CheckList:
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.