feat(csharp): fix Phase 1 Thrift telemetry gaps with E2E test coverage#345

Open

jadewang-db wants to merge 23 commits intomainfrom

stack/pr-phase1-thrift-telemetry-gaps

Commits on Mar 13, 2026

docs: add fix-telemetry-gaps design document

Jade Wang
and
claude
committed
docs: move E2E tests to Phase 1 (test-first approach)

Jade Wang
and
claude
committed
docs: consolidate implementation into 2 phases (Thrift gaps first, SEA second)
Jade Wang
committed
Build E2E test infrastructure with CapturingTelemetryExporter\n\nTask ID: task-1.1-e2e-test-infrastructure
Jade Wang
committed
Populate runtime_vendor and client_app_name in DriverSystemConfiguration\n\nTask ID: task-1.2-system-config-missing-fields
Jade Wang
committed
Populate auth_type on root telemetry log\n\nTask ID: task-1.3-auth-type-root-log
Jade Wang
committed
Populate WorkspaceId in TelemetrySessionContext\n\nTask ID: task-1.4-workspace-id
Jade Wang
committed
Expand DriverConnectionParameters with additional fields\n\nTask ID: task-1.5-connection-params-extended
Jade Wang
committed
Add ChunkMetrics aggregation to CloudFetchDownloader\n\nTask ID: task-1.6-chunk-metrics-aggregation
Jade Wang
committed
Expose GetChunkMetrics() on CloudFetchReader interface\n\nTask ID: task-1.7-expose-chunk-metrics-reader
Jade Wang
committed
Call SetChunkDetails() in DatabricksStatement.EmitTelemetry()\n\nTask ID: task-1.8-call-set-chunk-details
Jade Wang
committed
Track retry_count in SqlExecutionEvent\n\nTask ID: task-1.9-track-retry-count
Jade Wang
committed
Mark internal calls with is_internal_call flag\n\nTask ID: task-1.10-track-internal-calls
Jade Wang
committed
Add telemetry for metadata operations (GetObjects, GetTableTypes)\n\nTask ID: task-1.11-metadata-operation-telemetry
Jade Wang
committed
Verify all Phase 1 E2E tests pass\n\nTask ID: task-1.12-verify-phase1-tests
Jade Wang
committed
fix(csharp): remove accidentally committed demo submodule breaking CI
Jade Wang
committed
fix(csharp): fix lint issues - trailing whitespace, license headers, remove scratch files
Jade Wang
committed
fix(csharp): remove extra trailing newline in telemetry-design.md
Jade Wang
committed
fix(csharp): address PR review feedback - test isolation, assertions, resource cleanup
Jade Wang
committed

Commits on Mar 17, 2026

address comments
Jade Wang
committed
address pr comments
Jade Wang
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(csharp): fix Phase 1 Thrift telemetry gaps with E2E test coverage#345

feat(csharp): fix Phase 1 Thrift telemetry gaps with E2E test coverage#345
jadewang-db wants to merge 23 commits intomainfrom
stack/pr-phase1-thrift-telemetry-gaps

Commits on Mar 13, 2026

docs: add fix-telemetry-gaps design document

docs: move E2E tests to Phase 1 (test-first approach)

docs: consolidate implementation into 2 phases (Thrift gaps first, SEA second)

Build E2E test infrastructure with CapturingTelemetryExporter\n\nTask ID: task-1.1-e2e-test-infrastructure

Populate runtime_vendor and client_app_name in DriverSystemConfiguration\n\nTask ID: task-1.2-system-config-missing-fields

Populate auth_type on root telemetry log\n\nTask ID: task-1.3-auth-type-root-log

Populate WorkspaceId in TelemetrySessionContext\n\nTask ID: task-1.4-workspace-id

Expand DriverConnectionParameters with additional fields\n\nTask ID: task-1.5-connection-params-extended

Add ChunkMetrics aggregation to CloudFetchDownloader\n\nTask ID: task-1.6-chunk-metrics-aggregation

Expose GetChunkMetrics() on CloudFetchReader interface\n\nTask ID: task-1.7-expose-chunk-metrics-reader

Call SetChunkDetails() in DatabricksStatement.EmitTelemetry()\n\nTask ID: task-1.8-call-set-chunk-details

Track retry_count in SqlExecutionEvent\n\nTask ID: task-1.9-track-retry-count

Mark internal calls with is_internal_call flag\n\nTask ID: task-1.10-track-internal-calls

Add telemetry for metadata operations (GetObjects, GetTableTypes)\n\nTask ID: task-1.11-metadata-operation-telemetry

Verify all Phase 1 E2E tests pass\n\nTask ID: task-1.12-verify-phase1-tests

fix(csharp): remove accidentally committed demo submodule breaking CI

fix(csharp): fix lint issues - trailing whitespace, license headers, remove scratch files

fix(csharp): remove extra trailing newline in telemetry-design.md

fix(csharp): address PR review feedback - test isolation, assertions, resource cleanup

Commits on Mar 17, 2026

address comments

address pr comments

Commits on Mar 18, 2026

fix(csharp): address PR review - use existing constants, extract org ID from response header

fix(csharp): emit correct metadata telemetry for statement-level metadata commands