agent-substrate / agent-substrate/substrate

atepg: wrap request path transactions in one span

Đang mở
#1,537 0 bình luận 0 reaction 1 người được giao Được @git286 nhận Xem trên GitHub
area/observability kind/feature
Ngôn ngữ chính
Go
Star
1.8k
Fork
316
Merge trung bình
2 ngày 43 phút
Pull request đã merge (30 ngày)
287

Mô tả

Follow up from #1462.

The store tracer emits one client span per PostgreSQL statement. pgx sends BEGIN and COMMIT as normal statements, so a transaction on the request path shows as separate begin, statement and commit spans next to each other under the RPC span. This is correct but a bit noisy.

Idea from the review: wrap each transaction in one INTERNAL span and nest the statement spans under it. pgx has no hook per transaction, so this needs a small helper around pool.Begin in the store. Three call sites in cmd/ateapi/internal/store/atepg/atepg.go run on the request path. The outbox transactions run in background work without a parent span and produce no spans today.

We keep the begin and commit spans in any case. COMMIT is where the WAL flush time lands, so it carries real latency.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.