Skip to content

Feature/log rate limit - #90

Merged
JasmineJ1230 merged 4 commits into
masterfrom
feature/log_rate_limit
Jan 26, 2022
Merged

Feature/log rate limit#90
JasmineJ1230 merged 4 commits into
masterfrom
feature/log_rate_limit

Conversation

@JasmineJ1230

Copy link
Copy Markdown
Contributor

Description

Please explain the changes you've made

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #[issue number]

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

chenyijiang and others added 2 commits January 23, 2022 22:28
… information too many times in a period.

Feature2: Rollback to the default output log level after the config changed for a while.
Feature3: Format the namespace of metrics.
@codecov-commenter

codecov-commenter commented Jan 23, 2022

Copy link
Copy Markdown

Codecov Report

Merging #90 (e94bef7) into master (458717b) will decrease coverage by 1.26%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #90      +/-   ##
============================================
- Coverage     33.14%   31.88%   -1.27%     
  Complexity       69       69              
============================================
  Files            25       25              
  Lines           706      734      +28     
  Branches         69       73       +4     
============================================
  Hits            234      234              
- Misses          458      486      +28     
  Partials         14       14              
Impacted Files Coverage Δ
...pa/spi/aws/telemetry/metrics/CollectedMetrics.java 0.00% <0.00%> (ø)
.../spi/aws/telemetry/AwsCapaTelemetryProperties.java 0.00% <0.00%> (ø)
...s/telemetry/metrics/CloudWatchMetricsExporter.java 0.00% <0.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 458717b...e94bef7. Read the comment docs.

chenyijiang and others added 2 commits January 25, 2022 23:50
2.Change the format of all the common tag key. (eg. _aa_bb)
3.Delete unused code.
@JasmineJ1230
JasmineJ1230 merged commit 1c08e33 into master Jan 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants