Skip to content

Commit c55fe99

Browse files
committed
refactor StatsReporter
1 parent e558d3e commit c55fe99

File tree

2 files changed

+10
-7
lines changed

2 files changed

+10
-7
lines changed

packages/database/src/core/Repo.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ import {
5959
statsManagerGetCollection,
6060
statsManagerGetOrCreateReporter
6161
} from './stats/StatsManager';
62-
import { StatsReporter } from './stats/StatsReporter';
62+
import { StatsReporter, statsReporterIncludeStat } from './stats/StatsReporter';
6363
import { StatsListener } from './stats/StatsListener';
6464
import {
6565
EventQueue,
@@ -776,7 +776,7 @@ export function repoStats(repo: Repo, showDelta: boolean = false): void {
776776

777777
export function repoStatsIncrementCounter(repo: Repo, metric: string): void {
778778
repo.stats_.incrementCounter(metric);
779-
repo.statsReporter_.includeStat(metric);
779+
statsReporterIncludeStat(repo.statsReporter_, metric);
780780
}
781781

782782
function repoLog(repo: Repo, ...varArgs: unknown[]): void {

packages/database/src/core/stats/StatsReporter.ts

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ const REPORT_STATS_INTERVAL = 5 * 60 * 1000;
3232

3333
export class StatsReporter {
3434
private statsListener_: StatsListener;
35-
private statsToReport_: { [k: string]: boolean } = {};
35+
statsToReport_: { [k: string]: boolean } = {};
3636

3737
/**
3838
* @param collection
@@ -47,10 +47,6 @@ export class StatsReporter {
4747
setTimeoutNonBlocking(this.reportStats_.bind(this), Math.floor(timeout));
4848
}
4949

50-
includeStat(stat: string) {
51-
this.statsToReport_[stat] = true;
52-
}
53-
5450
private reportStats_() {
5551
const stats = this.statsListener_.get();
5652
const reportedStats: typeof stats = {};
@@ -74,3 +70,10 @@ export class StatsReporter {
7470
);
7571
}
7672
}
73+
74+
export function statsReporterIncludeStat(
75+
reporter: StatsReporter,
76+
stat: string
77+
) {
78+
reporter.statsToReport_[stat] = true;
79+
}

0 commit comments

Comments
 (0)