修复任务流连接错误上报及 ClickHouse 异常标记误判 - #1107
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
问题原因
任务第 9 轮有业务日志正文包含
__exception__。clickhouse-go v2.45.0的 HTTP Native 读取器在流正常结束时,会在整个响应中裸匹配该字符串,并将业务数据误判为 ClickHouse 异常块,最终稳定返回:clickhouse-go v2.48.0已修复该精确场景:正常 EOF 不再进入异常检测,并且只识别带协议 framing 的异常块。变更说明
attachStream的完整错误链,并通过任务 WebSocket 返回真实原因clickhouse-go/v2从 v2.45.0 升级到 v2.48.0,修复业务数据包含__exception__时的误判otelhttp移除DefaultClient的 API 变化,保持原有默认链路追踪行为验证
go test ./pkg/request ./pkg/clickhouse ./pkg/tasklog ./biz/task/handler/v1go test -race ./pkg/request ./pkg/clickhouse ./pkg/tasklog ./biz/task/handler/v1go vet ./pkg/request ./pkg/clickhouse ./pkg/tasklog ./biz/task/handler/v1go mod verify已知基线问题
go test ./...仍会因主分支已有测试桩缺少UpdateAgentResourceSelection方法而编译失败;已在未修改的main上复现,与本 PR 无关。