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

upgrade rcf libaries #239

Merged
merged 1 commit into from
Oct 3, 2020
Merged

upgrade rcf libaries #239

merged 1 commit into from
Oct 3, 2020

Conversation

wnbts
Copy link
Contributor

@wnbts wnbts commented Oct 2, 2020

This pr upgrades rcf libraries to the current version.

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

@wnbts wnbts marked this pull request as ready for review October 2, 2020 21:36
@codecov
Copy link

codecov bot commented Oct 2, 2020

Codecov Report

Merging #239 into master will increase coverage by 0.41%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #239      +/-   ##
============================================
+ Coverage     72.24%   72.66%   +0.41%     
- Complexity     1304     1330      +26     
============================================
  Files           146      149       +3     
  Lines          6043     6101      +58     
  Branches        476      480       +4     
============================================
+ Hits           4366     4433      +67     
+ Misses         1463     1450      -13     
- Partials        214      218       +4     
Flag Coverage Δ Complexity Δ
#cli 78.11% <ø> (ø) 0.00 <ø> (ø)
#plugin 72.03% <ø> (+0.46%) 1330.00 <ø> (+26.00)

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

Impacted Files Coverage Δ Complexity Δ
...stroforelasticsearch/ad/util/RestHandlerUtils.java 96.96% <0.00%> (-3.04%) 13.00% <0.00%> (ø%)
...stroforelasticsearch/ad/AnomalyDetectorPlugin.java 93.54% <0.00%> (ø) 10.00% <0.00%> (ø%)
...ransport/DeleteAnomalyDetectorTransportAction.java 59.03% <0.00%> (ø) 16.00% <0.00%> (?%)
...arch/ad/transport/DeleteAnomalyDetectorAction.java 100.00% <0.00%> (ø) 2.00% <0.00%> (?%)
...rch/ad/transport/DeleteAnomalyDetectorRequest.java 100.00% <0.00%> (ø) 6.00% <0.00%> (?%)
...sticsearch/ad/indices/AnomalyDetectionIndices.java 62.31% <0.00%> (+0.72%) 24.00% <0.00%> (+1.00%)
...asticsearch/ad/cluster/ADClusterEventListener.java 92.00% <0.00%> (+4.00%) 14.00% <0.00%> (+1.00%)
...earch/ad/rest/RestDeleteAnomalyDetectorAction.java 53.84% <0.00%> (+37.77%) 4.00% <0.00%> (ø%)

@@ -298,8 +298,8 @@ dependencies {
compile group: 'com.yahoo.datasketches', name: 'sketches-core', version: '0.13.4'
compile group: 'com.yahoo.datasketches', name: 'memory', version: '0.12.2'
compile group: 'commons-lang', name: 'commons-lang', version: '2.6'
compile 'software.amazon.randomcutforest:randomcutforest-core:1.0-alpha'
compile 'software.amazon.randomcutforest:randomcutforest-serialization-json:1.0-alpha'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Double check: 1.0 includes aws/random-cut-forest-by-aws@7b27ed9, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@wnbts wnbts merged commit 502c8c5 into opendistro-for-elasticsearch:master Oct 3, 2020
@ohltyler ohltyler added the version upgrade upgrade version like ES, java label Oct 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
version upgrade upgrade version like ES, java
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants