Skip to content

sql: add a tracing tag with the txn ID#74871

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
andreimatei:tracing.cleanup.sql-txn
Jan 21, 2022
Merged

sql: add a tracing tag with the txn ID#74871
craig[bot] merged 1 commit intocockroachdb:masterfrom
andreimatei:tracing.cleanup.sql-txn

Conversation

@andreimatei
Copy link
Copy Markdown
Contributor

This patch adds the txn's ID as a tag to the tracing span representing a
SQL txn. I'm creating a UI to explore the current spans, and this ID
will make it easy to navigate between a query/request blocking on a lock
held by some other txn, and the activity of that other txn.

Release note: None

This patch adds the txn's ID as a tag to the tracing span representing a
SQL txn. I'm creating a UI to explore the current spans, and this ID
will make it easy to navigate between a query/request blocking on a lock
held by some other txn, and the activity of that other txn.

Release note: None
@andreimatei andreimatei requested a review from rafiss January 14, 2022 22:05
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

Copy link
Copy Markdown
Contributor Author

@andreimatei andreimatei left a comment

Choose a reason for hiding this comment

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

bors r+

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained

@craig craig bot merged commit 488a06b into cockroachdb:master Jan 21, 2022
@craig
Copy link
Copy Markdown
Contributor

craig bot commented Jan 21, 2022

Build succeeded:

@andreimatei andreimatei deleted the tracing.cleanup.sql-txn branch January 21, 2022 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants