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

Implement isMuted method in DummyAction #355

Merged
merged 1 commit into from
Aug 7, 2020
Merged

Implement isMuted method in DummyAction #355

merged 1 commit into from
Aug 7, 2020

Conversation

ktkrg
Copy link
Contributor

@ktkrg ktkrg commented Aug 7, 2020

Issue #:
Fixes #354

Description of changes:
Fixes the build failure caused by DummyAction not implementing isMuted() now.

Tests:
Build is passing.

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

@ktkrg ktkrg added the bug Something isn't working label Aug 7, 2020
@ktkrg ktkrg requested review from yojs and rguo-aws August 7, 2020 19:24
@codecov
Copy link

codecov bot commented Aug 7, 2020

Codecov Report

Merging #355 into master will decrease coverage by 0.15%.
The diff coverage is 59.39%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #355      +/-   ##
============================================
- Coverage     67.28%   67.13%   -0.16%     
- Complexity     2051     2079      +28     
============================================
  Files           298      300       +2     
  Lines         13168    13311     +143     
  Branches       1087     1103      +16     
============================================
+ Hits           8860     8936      +76     
- Misses         3914     3969      +55     
- Partials        394      406      +12     
Impacted Files Coverage Δ Complexity Δ
...manceanalyzer/decisionmaker/deciders/Collator.java 30.00% <ø> (ø) 1.00 <0.00> (ø)
...rmanceanalyzer/decisionmaker/deciders/Decider.java 18.18% <ø> (ø) 2.00 <0.00> (ø)
...h/performanceanalyzer/rca/RcaControllerHelper.java 60.97% <0.00%> (-1.53%) 7.00 <0.00> (ø)
...formanceanalyzer/rca/framework/util/RcaConsts.java 88.88% <ø> (ø) 1.00 <0.00> (ø)
...erformanceanalyzer/rca/framework/core/RcaConf.java 44.03% <19.64%> (-22.12%) 23.00 <4.00> (+2.00) ⬇️
.../performanceanalyzer/rca/framework/core/Stats.java 90.19% <20.00%> (-9.81%) 22.00 <0.00> (-1.00)
...csearch/performanceanalyzer/rca/RcaController.java 80.43% <63.15%> (-0.47%) 37.00 <7.00> (ø)
...lyzer/config/overrides/ConfigOverridesApplier.java 72.72% <72.72%> (ø) 13.00 <13.00> (?)
...eanalyzer/reader/ClusterDetailsEventProcessor.java 71.00% <91.66%> (+2.46%) 16.00 <2.00> (+2.00)
.../elasticsearch/performanceanalyzer/AppContext.java 93.33% <100.00%> (+0.65%) 16.00 <2.00> (+2.00)
... and 16 more

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 7ede0b9...2d80a43. Read the comment docs.

@ktkrg ktkrg merged commit f9dfda5 into master Aug 7, 2020
@ktkrg ktkrg deleted the ktkrg-build-fix branch August 7, 2020 19:34
khushbr pushed a commit that referenced this pull request Sep 30, 2020
Changes for 1 package (OpenDistroPerformanceAnalyzerEngine), pushed in snapshot...

  https://code.amazon.com/snapshots/partsrut/2020-08-08T02-49-04

Changes for OpenDistroPerformanceAnalyzerEngine package:

0194f9f Fix failing reaction wheel tests
bfdb93d Merge remote-tracking branch 'upstream/master'
e03a120 Use StringUtils instead of NumberUtils to check timestamp string (#360)
79841f2 Expose queue rejection time period setting in rca.conf (#356)
f9dfda5 Implement isMuted method in DummyAction (#355)
7ede0b9 Reader changes for dynamic enable/disable of RCA graph components (#325)
ee58207 Fix bug in publisher to support cool off period on a per node basis (#351)

cr https://code.amazon.com/reviews/CR-31344155
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Build is failing
3 participants