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

remove deprecated code #228

Merged
merged 1 commit into from
Sep 14, 2020
Merged

remove deprecated code #228

merged 1 commit into from
Sep 14, 2020

Conversation

wnbts
Copy link
Member

@wnbts wnbts commented Sep 11, 2020

Description of changes: remove unused deprecated code in model manager.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@codecov
Copy link

codecov bot commented Sep 11, 2020

Codecov Report

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

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #228      +/-   ##
============================================
- Coverage     72.40%   72.20%   -0.20%     
+ Complexity     1289     1276      -13     
============================================
  Files           139      139              
  Lines          6073     6045      -28     
  Branches        469      469              
============================================
- Hits           4397     4365      -32     
- Misses         1464     1466       +2     
- Partials        212      214       +2     
Flag Coverage Δ Complexity Δ
#cli 80.30% <ø> (ø) 0.00 <ø> (ø)
#plugin 71.12% <ø> (-0.23%) 1276.00 <ø> (-13.00)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ Complexity Δ
...opendistroforelasticsearch/ad/ml/ModelManager.java 90.93% <ø> (-1.13%) 100.00 <0.00> (-12.00)
...asticsearch/ad/cluster/ADClusterEventListener.java 88.00% <0.00%> (-4.00%) 13.00% <0.00%> (-1.00%)

@wnbts wnbts marked this pull request as ready for review September 11, 2020 02:00
@wnbts wnbts merged commit 0fbfeac into opendistro-for-elasticsearch:master Sep 14, 2020
@wnbts wnbts deleted the clean branch September 14, 2020 20:29
@ohltyler ohltyler added the enhancement New feature or request label Oct 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants