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

Conversation

@dbbaughe
Copy link
Contributor

@dbbaughe dbbaughe commented Jul 6, 2020

Issue #, if available:

Description of changes:

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

@dbbaughe dbbaughe requested review from bowenlan-amzn and qreshi July 6, 2020 21:58
@codecov
Copy link

codecov bot commented Jul 6, 2020

Codecov Report

Merging #248 into master will decrease coverage by 0.10%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #248      +/-   ##
============================================
- Coverage     71.67%   71.56%   -0.11%     
  Complexity      633      633              
============================================
  Files            86       86              
  Lines          3647     3647              
  Branches        601      601              
============================================
- Hits           2614     2610       -4     
- Misses          722      726       +4     
  Partials        311      311              
Impacted Files Coverage Δ Complexity Δ
...ement/step/notification/AttemptNotificationStep.kt 62.06% <0.00%> (-20.69%) 5.00% <0.00%> (ø%)
...ticsearch/indexstatemanagement/model/Transition.kt 80.39% <0.00%> (-1.97%) 5.00% <0.00%> (ø%)
...ndexstatemanagement/IndexStateManagementIndices.kt 66.66% <0.00%> (+6.24%) 10.00% <0.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 49902de...9c48c7c. Read the comment docs.

@dbbaughe
Copy link
Contributor Author

dbbaughe commented Jul 6, 2020

Codecov failure because of testing randomization, unrelated to PR.

@dbbaughe dbbaughe merged commit f7b193c into opendistro-for-elasticsearch:master Jul 6, 2020
@bowenlan-amzn bowenlan-amzn added the documentation A change to update existing documentations such as README, instruction docs, etc. label Jul 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

documentation A change to update existing documentations such as README, instruction docs, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants