Skip to content

Commit dcdc5e9

Browse files
authored
fix(rds): switch connection monitor to average (#233)
Fixes #231 --- _By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license_
1 parent 087bb1c commit dcdc5e9

File tree

4 files changed

+11
-11
lines changed

4 files changed

+11
-11
lines changed

lib/monitoring/aws-rds/RdsClusterMetricFactory.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ export class RdsClusterMetricFactory {
5454
metricTotalConnectionCount() {
5555
return this.metric(
5656
"DatabaseConnections",
57-
MetricStatistic.SUM,
57+
MetricStatistic.AVERAGE,
5858
"Connections"
5959
);
6060
}

test/facade/__snapshots__/MonitoringAspect.test.ts.snap

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

test/monitoring/aws-rds/__snapshots__/RdsClusterMonitoring.test.ts.snap

Lines changed: 4 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

test/monitoring/aws-redshift/__snapshots__/RedshiftClusterMonitoring.test.ts.snap

Lines changed: 4 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)