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

upgrade to ES 7.10.2 #378

Merged
merged 1 commit into from
Feb 1, 2021

Conversation

ylwu-amzn
Copy link
Contributor

Issue #, if available:

Description of changes:

Upgrade to ES 7.10.2

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 Feb 1, 2021

Codecov Report

Merging #378 (b1dc59e) into master (38cb557) will decrease coverage by 0.17%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #378      +/-   ##
============================================
- Coverage     79.20%   79.02%   -0.18%     
+ Complexity     2665     2658       -7     
============================================
  Files           247      247              
  Lines         11726    11726              
  Branches       1009     1009              
============================================
- Hits           9288     9267      -21     
- Misses         1961     1976      +15     
- Partials        477      483       +6     
Flag Coverage Δ Complexity Δ
plugin 79.01% <ø> (-0.20%) 0.00 <ø> (ø)

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

Impacted Files Coverage Δ Complexity Δ
...port/SearchAnomalyDetectorInfoTransportAction.java 54.54% <0.00%> (-9.10%) 4.00% <0.00%> (ø%)
...stroforelasticsearch/ad/AnomalyDetectorRunner.java 40.00% <0.00%> (-6.25%) 6.00% <0.00%> (-1.00%)
...pendistroforelasticsearch/ad/cluster/HashRing.java 78.94% <0.00%> (-5.27%) 10.00% <0.00%> (-1.00%)
...on/opendistroforelasticsearch/ad/model/ADTask.java 95.31% <0.00%> (-2.82%) 64.00% <0.00%> (-4.00%)
...ndistroforelasticsearch/ad/task/ADTaskManager.java 72.82% <0.00%> (-0.23%) 81.00% <0.00%> (-1.00%)

Copy link
Contributor

@saratvemulapalli saratvemulapalli left a comment

Choose a reason for hiding this comment

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

Just curious: Do you know what changed in job scheduler, the binary file size bumped up more than 3x?

Also are you taking care of the release notes in a different PR?

compileOnly "com.amazon.opendistroforelasticsearch:opendistro-job-scheduler-spi:1.12.0.0"
compile "com.amazon.opendistroforelasticsearch:common-utils:1.12.0.2"
compileOnly "com.amazon.opendistroforelasticsearch:opendistro-job-scheduler-spi:${opendistroVersion}.0"
compile "com.amazon.opendistroforelasticsearch:common-utils:${opendistroVersion}.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice, I like this!

Copy link
Contributor

@ohltyler ohltyler left a comment

Choose a reason for hiding this comment

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

Thanks for handling the upgrade!

@ylwu-amzn ylwu-amzn merged commit 8042704 into opendistro-for-elasticsearch:master Feb 1, 2021
@ohltyler ohltyler added the version upgrade upgrade version like ES, java label Feb 2, 2021
ylwu-amzn added a commit that referenced this pull request Feb 24, 2021
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.

3 participants