Skip to content

fix(database_observability.mysql): Skip wait events with NULL timer_wait#5478

Merged
cristiangreco merged 1 commit intomainfrom
cristian/dbo11y-mysql-wait-event-valid
Feb 12, 2026
Merged

fix(database_observability.mysql): Skip wait events with NULL timer_wait#5478
cristiangreco merged 1 commit intomainfrom
cristian/dbo11y-mysql-wait-event-valid

Conversation

@cristiangreco
Copy link
Contributor

@cristiangreco cristiangreco commented Feb 9, 2026

Brief description of Pull Request

Do not log wait events with NULL timer_wait (currently logged as "0ms" duration), as they do not provide meaningful information.

Pull Request Details

Avoid logs like

wait_event_name="wait/io/socket/sql/client_connection" wait_object_type="SOCKET" wait_time="0.000000ms" sql_text="COMMIT" 

Issue(s) fixed by this Pull Request

Notes to the Reviewer

PR Checklist

  • Documentation added
  • Tests updated
  • Config converters updated

Do not log wait events with NULL timer_wait (currently logged as "0ms"
duration), as they do not provide meaningful information.
@cristiangreco cristiangreco changed the title fix(database_observability.mysql): skip wait events with NULL timer_wait fix(database_observability.mysql): Skip wait events with NULL timer_wait Feb 9, 2026
@cristiangreco cristiangreco marked this pull request as ready for review February 9, 2026 13:37
@cristiangreco cristiangreco requested a review from a team as a code owner February 9, 2026 13:37
@cristiangreco cristiangreco merged commit 48750e5 into main Feb 12, 2026
48 of 49 checks passed
@cristiangreco cristiangreco deleted the cristian/dbo11y-mysql-wait-event-valid branch February 12, 2026 10:59
@grafana-alloybot grafana-alloybot bot mentioned this pull request Feb 12, 2026
@cristiangreco cristiangreco added the backport/v1.13 Backport to release/v1.13 label Feb 12, 2026
grafana-alloybot bot pushed a commit that referenced this pull request Feb 12, 2026
…_wait (#5478)

### Brief description of Pull Request
Do not log wait events with NULL timer_wait (currently logged as "0ms"
duration), as they do not provide meaningful information.

### Pull Request Details

Avoid logs like

```
wait_event_name="wait/io/socket/sql/client_connection" wait_object_type="SOCKET" wait_time="0.000000ms" sql_text="COMMIT"
```

### Issue(s) fixed by this Pull Request

<!--
Uncomment the following line and fill in an issue number if you want a
GitHub
  issue to be closed automatically when this PR gets merged.
-->

<!-- Fixes #issue_id -->

### Notes to the Reviewer

<!-- Add any relevant notes for the reviewers and testers of this PR.
-->

### PR Checklist

<!-- Remove items that do not apply. For completed items, change [ ] to
[x]. -->

- [ ] Documentation added
- [x] Tests updated
- [ ] Config converters updated

(cherry picked from commit 48750e5)
cristiangreco added a commit that referenced this pull request Feb 12, 2026
…_wait [backport] (#5521)

## Backport of #5478

This PR backports #5478 to release/v1.13.

### Original PR Author
@cristiangreco

### Description
### Brief description of Pull Request
Do not log wait events with NULL timer_wait (currently logged as "0ms"
duration), as they do not provide meaningful information.

### Pull Request Details

Avoid logs like

```
wait_event_name="wait/io/socket/sql/client_connection" wait_object_type="SOCKET" wait_time="0.000000ms" sql_text="COMMIT" 
```

### Issue(s) fixed by this Pull Request

<!--
Uncomment the following line and fill in an issue number if you want a
GitHub
  issue to be closed automatically when this PR gets merged.
-->

<!-- Fixes #issue_id -->

### Notes to the Reviewer

<!-- Add any relevant notes for the reviewers and testers of this PR.
-->

### PR Checklist

<!-- Remove items that do not apply. For completed items, change [ ] to
[x]. -->

- [ ] Documentation added
- [x] Tests updated
- [ ] Config converters updated


---
*This backport was created automatically.*

Co-authored-by: Cristian Greco <cristian@regolo.cc>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 27, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backport/v1.13 Backport to release/v1.13 frozen-due-to-age

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants