Skip to content

Commit 70c038c

Browse files
samikshya-dbclaude
andcommitted
Move auth_type to top level per proto definition
- auth_type is field 5 at OssSqlDriverTelemetryLog level, not nested - Remove driver_connection_params (not populated in Node.js driver) - Export auth_type directly in sql_driver_log for connection events Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
1 parent 658870f commit 70c038c

1 file changed

Lines changed: 3 additions & 5 deletions

File tree

lib/telemetry/DatabricksTelemetryExporter.ts

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ interface DatabricksTelemetryLog {
5252
char_set_encoding?: string;
5353
process_name?: string;
5454
};
55-
driver_connection_params?: any;
55+
auth_type?: string;
5656
operation_latency_ms?: number;
5757
sql_operation?: {
5858
statement_type?: string;
@@ -301,10 +301,8 @@ export default class DatabricksTelemetryExporter {
301301
if (metric.latencyMs !== undefined) {
302302
log.entry.sql_driver_log.operation_latency_ms = metric.latencyMs;
303303
}
304-
// Include driver connection params (auth type)
305-
log.entry.sql_driver_log.driver_connection_params = {
306-
auth_type: metric.driverConfig.authType,
307-
};
304+
// Include auth type at top level (proto field 5)
305+
log.entry.sql_driver_log.auth_type = metric.driverConfig.authType;
308306
} else if (metric.metricType === 'statement') {
309307
log.entry.sql_driver_log.operation_latency_ms = metric.latencyMs;
310308

0 commit comments

Comments
 (0)