Skip to content

fix: Database_observability: update BackendXID type to int64 to better map to PG xid#5370

Merged
matthewnolf merged 2 commits intomainfrom
mn/db-o11y-psql-backendxid
Jan 28, 2026
Merged

fix: Database_observability: update BackendXID type to int64 to better map to PG xid#5370
matthewnolf merged 2 commits intomainfrom
mn/db-o11y-psql-backendxid

Conversation

@matthewnolf
Copy link
Contributor

Brief description of Pull Request

Pull Request Details

This change updates the integer type used for our representation of BackendXID from int32, to int64 which better maps to Postgres xid data type

pg_stat_activity view docs
xid docs

Issue(s) fixed by this Pull Request

Notes to the Reviewer

PR Checklist

  • Documentation added
  • Tests updated
  • Config converters updated

@matthewnolf matthewnolf requested a review from a team as a code owner January 28, 2026 09:39
@matthewnolf matthewnolf force-pushed the mn/db-o11y-psql-backendxid branch from 5f39bc3 to 227342b Compare January 28, 2026 09:48
@matthewnolf matthewnolf merged commit f5c20f5 into main Jan 28, 2026
47 checks passed
@matthewnolf matthewnolf deleted the mn/db-o11y-psql-backendxid branch January 28, 2026 12:33
@grafana-alloybot grafana-alloybot bot mentioned this pull request Jan 28, 2026
@ptodev ptodev added the backport/v1.13 Backport to release/v1.13 label Jan 28, 2026
grafana-alloybot bot pushed a commit that referenced this pull request Jan 28, 2026
…r map to PG xid (#5370)

<!--
  CONTRIBUTORS GUIDE:

https://github.com/grafana/alloy/blob/main/docs/developer/contributing.md

If this is your first PR or you have not contributed in a while, we
recommend
  taking the time to review the guide.

  **NOTE**
Your PR title must adhere to Conventional Commit style. For details on
this,
  check out the Contributors Guide linked above.
-->

### Brief description of Pull Request

<!--
Add a human-readable description of the PR that may be used as the
commit body
  (i.e. "Extended description") when it gets merged.
-->

### Pull Request Details
This change updates the integer type used for our representation of
BackendXID from int32, to int64 which better maps to Postgres xid data
type

[pg_stat_activity view
docs](https://www.postgresql.org/docs/current/monitoring-stats.html#MONITORING-PG-STAT-ACTIVITY-VIEW)
[xid docs](https://www.postgresql.org/docs/current/transaction-id.html)

<!-- Add a more detailed descripion of the Pull Request here, if needed.
-->

### 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
- [ ] Tests updated
- [ ] Config converters updated

(cherry picked from commit f5c20f5)
ptodev pushed a commit that referenced this pull request Jan 28, 2026
…r map to PG xid [backport] (#5373)

## Backport of #5370

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

### Original PR Author
@matthewnolf

### Description
<!--
  CONTRIBUTORS GUIDE:

https://github.com/grafana/alloy/blob/main/docs/developer/contributing.md

If this is your first PR or you have not contributed in a while, we
recommend
  taking the time to review the guide.

  **NOTE**
Your PR title must adhere to Conventional Commit style. For details on
this,
  check out the Contributors Guide linked above.
-->

### Brief description of Pull Request

<!--
Add a human-readable description of the PR that may be used as the
commit body
  (i.e. "Extended description") when it gets merged.
-->

### Pull Request Details
This change updates the integer type used for our representation of
BackendXID from int32, to int64 which better maps to Postgres xid data
type

[pg_stat_activity view
docs](https://www.postgresql.org/docs/current/monitoring-stats.html#MONITORING-PG-STAT-ACTIVITY-VIEW)
[xid docs](https://www.postgresql.org/docs/current/transaction-id.html)

<!-- Add a more detailed descripion of the Pull Request here, if needed.
-->

### 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
- [ ] Tests updated
- [ ] Config converters updated


---
*This backport was created automatically.*

Co-authored-by: MattNolf <86960799+matthewnolf@users.noreply.github.com>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 12, 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.

3 participants