Add debug logging for telemetry panic recovery

Enable debug-level logging for all panic recovery blocks in telemetry code.
This aids debugging while keeping logs clean (only visible with debug enabled).

Changes:
- Add logger import to exporter.go, aggregator.go, interceptor.go
- Enable debug logging in 6 panic recovery blocks:
  - exporter: export panic
  - aggregator: recordMetric, completeStatement, async export panics
  - interceptor: afterExecute, recordConnection panics

Addresses review feedback on PR #320

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: samikshya-db

telemetry/aggregator.go

@@ -4,6 +4,8 @@ import (
 	"context"
 	"sync"
 	"time"
+
+	"github.com/databricks/databricks-sql-go/logger"
 )
 
 // metricsAggregator aggregates metrics by statement and batches for export.
@@ -54,8 +56,7 @@ func (agg *metricsAggregator) recordMetric(ctx context.Context, metric *telemetr
 	// Swallow all errors
 	defer func() {
 		if r := recover(); r != nil {
-			// Log at trace level only
-			// logger.Trace().Msgf("telemetry: recordMetric panic: %v", r)
+			logger.Debug().Msgf("telemetry: recordMetric panic: %v", r)
 		}
 	}()
 
@@ -125,7 +126,7 @@ func (agg *metricsAggregator) recordMetric(ctx context.Context, metric *telemetr
 func (agg *metricsAggregator) completeStatement(ctx context.Context, statementID string, failed bool) {
 	defer func() {
 		if r := recover(); r != nil {
-			// Log at trace level only
+			logger.Debug().Msgf("telemetry: completeStatement panic: %v", r)
 		}
 	}()
 
@@ -204,7 +205,7 @@ func (agg *metricsAggregator) flushUnlocked(ctx context.Context) {
 	go func() {
 		defer func() {
 			if r := recover(); r != nil {
-				// Log at trace level only
+				logger.Debug().Msgf("telemetry: async export panic: %v", r)
 			}
 		}()
 		agg.exporter.export(ctx, metrics)

telemetry/exporter.go

@@ -8,6 +8,8 @@ import (
 	"net/http"
 	"strings"
 	"time"
+
+	"github.com/databricks/databricks-sql-go/logger"
 )
 
 // telemetryExporter exports metrics to Databricks telemetry service.
@@ -64,7 +66,7 @@ func (e *telemetryExporter) export(ctx context.Context, metrics []*telemetryMetr
 	defer func() {
 		if r := recover(); r != nil {
 			// Intentionally swallow panic - telemetry must not impact driver
-			_ = r // Log at trace level only: logger.Trace().Msgf("telemetry: export panic: %v", r)
+			logger.Debug().Msgf("telemetry: export panic: %v", r)
 		}
 	}()
 

telemetry/interceptor.go

@@ -3,6 +3,8 @@ package telemetry
 import (
 	"context"
 	"time"
+
+	"github.com/databricks/databricks-sql-go/logger"
 )
 
 // Interceptor wraps driver operations to collect metrics.
@@ -75,8 +77,7 @@ func (i *Interceptor) afterExecute(ctx context.Context, err error) {
 	// Swallow all panics
 	defer func() {
 		if r := recover(); r != nil {
-			// Log at trace level only
-			// logger.Trace().Msgf("telemetry: afterExecute panic: %v", r)
+			logger.Debug().Msgf("telemetry: afterExecute panic: %v", r)
 		}
 	}()
 
@@ -116,7 +117,7 @@ func (i *Interceptor) recordConnection(ctx context.Context, tags map[string]inte
 
 	defer func() {
 		if r := recover(); r != nil {
-			// Log at trace level only
+			logger.Debug().Msgf("telemetry: recordConnection panic: %v", r)
 		}
 	}()