Skip to content
This repository has been archived by the owner on Mar 27, 2021. It is now read-only.

add Series getter for hashCode, modify write cache conditions, enhance test #731

Merged
merged 2 commits into from
Dec 18, 2020

Conversation

adsail
Copy link
Contributor

@adsail adsail commented Dec 17, 2020

No description provided.

@codecov
Copy link

codecov bot commented Dec 17, 2020

Codecov Report

Merging #731 (fcf7e6b) into master (08a9405) will decrease coverage by 0.00%.
The diff coverage is 71.42%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #731      +/-   ##
============================================
- Coverage     54.95%   54.95%   -0.01%     
- Complexity     3104     3105       +1     
============================================
  Files           739      739              
  Lines         20143    20147       +4     
  Branches       1312     1313       +1     
============================================
+ Hits          11069    11071       +2     
- Misses         8607     8608       +1     
- Partials        467      468       +1     
Impacted Files Coverage Δ Complexity Δ
...roic/metadata/elasticsearch/MetadataBackendKV.java 81.74% <60.00%> (+0.21%) 50.00 <0.00> (+1.00)
...rc/main/java/com/spotify/heroic/common/Series.java 80.74% <100.00%> (+0.14%) 41.00 <1.00> (+1.00)
...c/test/AbstractMetadataBackendIndexResourceIT.java 94.47% <100.00%> (ø) 29.00 <0.00> (ø)
...com/spotify/heroic/aggregation/simple/MinBucket.kt 44.44% <0.00%> (-11.12%) 4.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 08a9405...fcf7e6b. Read the comment docs.

@adsail adsail merged commit 5f757f5 into master Dec 18, 2020
@delete-merged-branch delete-merged-branch bot deleted the hotfix-index-ri-cache branch December 18, 2020 17:58
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.

2 participants