Skip to content

Commit

Permalink
[pinpoint-apm#132] active trace is no needs. If ActiveCount enable, I…
Browse files Browse the repository at this point in the history
…t needs to be enable
  • Loading branch information
feelform committed Apr 27, 2023
1 parent 8c60602 commit 75863f1
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions lib/context/trace-context.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ const Trace = require('./trace')
const TransactionId = require('./transaction-id')
const TraceId = require('./trace-id')
const IdGenerator = require('./id-generator')
const activeTrace = require('../metric/active-trace')
const log = require('../utils/logger')
const sampler = require('../sampler/sampler')
const DisableTrace = require('./disable-trace')
Expand Down Expand Up @@ -78,7 +77,6 @@ class TraceContext {
try {
const trace = new Trace(traceId, this.agentInfo, this.dataSender, sampling, requestData)
this.setCurrentTraceObject(trace)
activeTrace.register(trace)
return trace
} catch (e) {
log.error('Fail to create trace object', e)
Expand All @@ -92,7 +90,6 @@ class TraceContext {
try {
trace.spanRecorder.span.markElapsedTime()
trace.close()
activeTrace.remove(trace)
} catch (e) {
log.error('Fail to complete trace object', e)
}
Expand Down

0 comments on commit 75863f1

Please sign in to comment.