Skip to content

Commit 57a740a

Browse files
committed
Remove cpu-experimental profiler
1 parent 2fed3ce commit 57a740a

5 files changed

Lines changed: 2 additions & 310 deletions

File tree

packages/dd-trace/src/profiling/config.js

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ const { URL, format, pathToFileURL } = require('url')
77
const { AgentExporter } = require('./exporters/agent')
88
const { FileExporter } = require('./exporters/file')
99
const { ConsoleLogger } = require('./loggers/console')
10-
const CpuProfiler = require('./profilers/cpu')
1110
const WallProfiler = require('./profilers/wall')
1211
const SpaceProfiler = require('./profilers/space')
1312
const { oomExportStrategies, snapshotKinds } = require('./constants')
@@ -202,8 +201,6 @@ function getProfiler (name, options) {
202201
return new WallProfiler(options)
203202
case 'space':
204203
return new SpaceProfiler(options)
205-
case 'cpu-experimental':
206-
return new CpuProfiler(options)
207204
default:
208205
options.logger.error(`Unknown profiler "${name}"`)
209206
}

packages/dd-trace/src/profiling/index.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
'use strict'
22

33
const { Profiler, ServerlessProfiler } = require('./profiler')
4-
const CpuProfiler = require('./profilers/cpu')
54
const WallProfiler = require('./profilers/wall')
65
const SpaceProfiler = require('./profilers/space')
76
const { AgentExporter } = require('./exporters/agent')
@@ -14,7 +13,6 @@ module.exports = {
1413
profiler,
1514
AgentExporter,
1615
FileExporter,
17-
CpuProfiler,
1816
WallProfiler,
1917
SpaceProfiler,
2018
ConsoleLogger

packages/dd-trace/src/profiling/profilers/cpu.js

Lines changed: 0 additions & 126 deletions
This file was deleted.

packages/dd-trace/test/profiling/config.spec.js

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ const os = require('os')
77
const path = require('path')
88
const { AgentExporter } = require('../../src/profiling/exporters/agent')
99
const { FileExporter } = require('../../src/profiling/exporters/file')
10-
const CpuProfiler = require('../../src/profiling/profilers/cpu')
1110
const WallProfiler = require('../../src/profiling/profilers/wall')
1211
const SpaceProfiler = require('../../src/profiling/profilers/space')
1312
const { ConsoleLogger } = require('../../src/profiling/loggers/console')
@@ -58,7 +57,7 @@ describe('config', () => {
5857
error () { }
5958
},
6059
exporters: 'agent,file',
61-
profilers: 'wall,cpu-experimental',
60+
profilers: 'wall',
6261
url: 'http://localhost:1234/'
6362
}
6463

@@ -79,9 +78,8 @@ describe('config', () => {
7978
expect(config.exporters[0]._url.toString()).to.equal(options.url)
8079
expect(config.exporters[1]).to.be.an.instanceof(FileExporter)
8180
expect(config.profilers).to.be.an('array')
82-
expect(config.profilers.length).to.equal(2)
81+
expect(config.profilers.length).to.equal(1)
8382
expect(config.profilers[0]).to.be.an.instanceOf(WallProfiler)
84-
expect(config.profilers[1]).to.be.an.instanceOf(CpuProfiler)
8583
})
8684

8785
it('should filter out invalid profilers', () => {

packages/dd-trace/test/profiling/profilers/cpu.spec.js

Lines changed: 0 additions & 175 deletions
This file was deleted.

0 commit comments

Comments
 (0)