Skip to content

Commit 3a83c09

Browse files
committed
feat: add batch size metrics to publish telemetry
1 parent f22813c commit 3a83c09

File tree

2 files changed

+3
-11
lines changed

2 files changed

+3
-11
lines changed

common/telemetry.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,6 @@ const batchMetricsWriteClient = influx.getWriteApi(
3535
setInterval(() => {
3636
publishWriteClient.flush().catch(console.error)
3737
networkInfoWriteClient.flush().catch(console.error)
38-
batchMetricsWriteClient.flush().catch(console.error)
3938
}, 10_000).unref()
4039

4140
const recordFn = (client, name, fn) => {
@@ -51,6 +50,5 @@ export {
5150
publishWriteClient,
5251
networkInfoWriteClient,
5352
recordPublishTelemetry,
54-
recordNetworkInfoTelemetry,
55-
batchMetricsWriteClient
53+
recordNetworkInfoTelemetry
5654
}

publish/index.js

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ export const publish = async ({
131131

132132
logger.log('Done!')
133133

134-
// Enhanced telemetry recording with separate batch metrics
134+
// Telemetry recording with batch size metrics
135135
recordTelemetry('publish', point => {
136136
// Existing metrics
137137
point.intField('round_index', roundIndex)
@@ -142,16 +142,10 @@ export const publish = async ({
142142
uploadMeasurementsDuration
143143
)
144144
point.intField('add_measurements_duration_ms', ieAddMeasurementsDuration)
145-
})
146145

147-
// Separate batch metrics recording for better organization
148-
recordTelemetry('batch_metrics', point => {
146+
// Add batch size metrics to existing publish point
149147
point.intField('batch_size_bytes', batchSizeBytes)
150148
point.floatField('avg_measurement_size_bytes', batchSizeBytes / measurements.length)
151-
point.intField('measurement_count', measurements.length)
152-
point.tag('cid', cid.toString())
153-
point.tag('round_index', roundIndex.toString())
154-
point.timestamp(new Date())
155149
})
156150
}
157151

0 commit comments

Comments
 (0)