Skip to content

Commit 0762cb7

Browse files
committed
update telemetry
1 parent 73b4210 commit 0762cb7

File tree

3 files changed

+9
-9
lines changed

3 files changed

+9
-9
lines changed

packages/core/src/shared/telemetry/vscodeTelemetry.json

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
{
22
"types": [
33
{
4-
"name": "size",
4+
"name": "numChildProcesses",
55
"type": "int",
6-
"description": "A generic size for when units are clear"
6+
"description": "number of child processes actively running"
77
},
88
{
99
"name": "childProcess",
@@ -389,11 +389,11 @@
389389
]
390390
},
391391
{
392-
"name": "ide_reportPerformance",
393-
"description": "Emitted when user invokes reportPerformance command",
392+
"name": "ide_showExtStats",
393+
"description": "Emitted when user invokes show extensions stats command",
394394
"metadata": [
395395
{
396-
"type": "size",
396+
"type": "numChildProcesses",
397397
"required": true
398398
}
399399
]

packages/core/src/shared/utilities/processUtils.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -187,8 +187,8 @@ export class ChildProcessTracker {
187187
}
188188

189189
public async logAllUsage(): Promise<void> {
190-
await telemetry.ide_logActiveProcesses.run(async (span) => {
191-
span.record({ size: this.size })
190+
await telemetry.ide_showExtStats.run(async (span) => {
191+
span.record({ numChildProcesses: this.size })
192192
if (this.size === 0) {
193193
this.logger.info('No Active Subprocesses')
194194
return

packages/core/src/test/shared/utilities/processUtils.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -546,7 +546,7 @@ describe('ChildProcessTracker', function () {
546546

547547
it('emits telemetry with size equal to number of processes (empty)', async function () {
548548
await tracker.logAllUsage()
549-
assertTelemetry('ide_logActiveProcesses', { size: 0 })
549+
assertTelemetry('ide_showExtStats', { numChildProcesses: 0 })
550550
})
551551

552552
it('emits telemetry to number of processes (nonempty)', async function () {
@@ -556,7 +556,7 @@ describe('ChildProcessTracker', function () {
556556
}
557557

558558
await tracker.logAllUsage()
559-
assertTelemetry('ide_logActiveProcesses', { size: size })
559+
assertTelemetry('ide_showExtStats', { numChildProcesses: size })
560560
})
561561
})
562562

0 commit comments

Comments
 (0)