Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Support dashboard filters on rollup indices #346

Merged
merged 2 commits into from
Nov 19, 2020

Conversation

thalurur
Copy link
Contributor

Issue #, if available:
N/A

Description of changes:
Adding support to basic match phrase queries

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@codecov
Copy link

codecov bot commented Nov 19, 2020

Codecov Report

Merging #346 (2d104c2) into master (46d1fce) will decrease coverage by 0.03%.
The diff coverage is 70.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #346      +/-   ##
============================================
- Coverage     74.96%   74.92%   -0.04%     
  Complexity     1348     1348              
============================================
  Files           183      183              
  Lines          6962     6972      +10     
  Branches       1129     1133       +4     
============================================
+ Hits           5219     5224       +5     
- Misses         1142     1144       +2     
- Partials        601      604       +3     
Impacted Files Coverage Δ Complexity Δ
...management/rollup/interceptor/RollupInterceptor.kt 68.96% <40.00%> (-1.31%) 31.00 <0.00> (+1.00) ⬇️
...csearch/indexmanagement/rollup/util/RollupUtils.kt 76.66% <100.00%> (+0.49%) 0.00 <0.00> (ø)
...ndexstatemanagement/IndexStateManagementHistory.kt 75.96% <0.00%> (-1.93%) 27.00% <0.00%> (-1.00%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 46d1fce...7881368. Read the comment docs.

@thalurur thalurur merged commit 423a825 into opendistro-for-elasticsearch:master Nov 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants