Skip to content

fix: update kernel pin for final-session telemetry flush - #523

Merged
vuanhphung merged 1 commit into
mainfrom
vu-phung_data/stack/kernel-final-session-telemetry-flush
Oct 30, 2026
Merged

vuanhphung merged 1 commit into
mainfrom
vu-phung_data/stack/kernel-final-session-telemetry-flush

Conversation

@vuanhphung

Copy link
Copy Markdown
Collaborator

Updates the source-build kernel pin from d72c264 to b5e545a, the merge commit for databricks/databricks-sql-kernel#413. Closing the first of two sessions now preserves the shared telemetry buffer until the final session closes, addressing the TELEMETRY-015 failure in the Go SEA nightly.

Validation: built the pinned kernel and ran the kernel-tagged Go unit tests with make test-kernel; git diff --check.


This PR was created with GitHub MCP.

Pin the source-built kernel to the merge of databricks/databricks-sql-kernel#413 so closing one session does not flush another session’s telemetry.

Signed-off-by: Vu Anh Phung <vu.phung@databricks.com>
@vuanhphung vuanhphung added ai-assisted integration-test-kernel Preview the full passthrough kernel integration suite (live warehouse) labels Oct 30, 2026
@github-actions

Copy link
Copy Markdown

Go integration tests triggered (sea / passthrough). View workflow runs.

@peco-review-bot peco-review-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ No issues identified by the review bot.

@vuanhphung
vuanhphung enabled auto-merge October 30, 2026 02:53
@vuanhphung
vuanhphung added this pull request to the merge queue Oct 30, 2026
Merged via the queue into main with commit a9a5c24 Oct 30, 2026
21 of 22 checks passed
@vuanhphung
vuanhphung deleted the vu-phung_data/stack/kernel-final-session-telemetry-flush branch October 30, 2026 03:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-assisted integration-test-kernel Preview the full passthrough kernel integration suite (live warehouse)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant