Skip to content

Comments

perf(monitor): Fold max_* into combo queries#96

Merged
smudge merged 1 commit intomainfrom
smudge/stack/7
Feb 12, 2026
Merged

perf(monitor): Fold max_* into combo queries#96
smudge merged 1 commit intomainfrom
smudge/stack/7

Conversation

@smudge
Copy link
Member

@smudge smudge commented Feb 10, 2026

Stacked PRs:


perf(monitor): Fold max_* into combo queries

This folds max_lock_age and max_age (and by extension,
age_alert_percent) into the combined monitor queries introduced in the
prior commit.

As before, I split these into two aggregate queries, hinging on whether
locked_at (and therefore, a heap lookup) is required to return a
result.

/no-platform

This folds `max_lock_age` and `max_age` (and by extension,
`age_alert_percent`) into the combined monitor queries introduced in the
prior commit.

As before, I split these into two aggregate queries, hinging on whether
`locked_at` (and therefore, a heap lookup) is required to return a
result.

/no-platform

stack-info: PR: #96, branch: smudge/stack/7
@smudge smudge marked this pull request as draft February 10, 2026 17:26
@smudge smudge changed the base branch from smudge/stack/6 to main February 10, 2026 17:26
@smudge smudge changed the base branch from main to smudge/stack/6 February 10, 2026 17:27
@smudge smudge marked this pull request as ready for review February 10, 2026 17:27
@smudge smudge marked this pull request as draft February 10, 2026 17:28
@smudge smudge changed the base branch from smudge/stack/6 to main February 10, 2026 17:28
@smudge smudge changed the base branch from main to smudge/stack/6 February 10, 2026 17:29
@smudge smudge mentioned this pull request Feb 10, 2026
@smudge smudge marked this pull request as ready for review February 10, 2026 17:29
@smudge smudge marked this pull request as draft February 10, 2026 17:31
@smudge smudge changed the base branch from smudge/stack/6 to main February 10, 2026 17:31
@smudge smudge changed the base branch from main to smudge/stack/6 February 10, 2026 17:32
@smudge smudge marked this pull request as ready for review February 10, 2026 17:32
@smudge smudge requested a review from effron February 10, 2026 17:37
@smudge smudge marked this pull request as draft February 10, 2026 18:10
@smudge smudge changed the base branch from smudge/stack/6 to main February 10, 2026 18:10
@smudge smudge changed the base branch from main to smudge/stack/6 February 10, 2026 18:10
@smudge smudge marked this pull request as ready for review February 10, 2026 18:11
smudge added a commit that referenced this pull request Feb 10, 2026
This folds `max_lock_age` and `max_age` (and by extension,
`age_alert_percent`) into the combined monitor queries introduced in the
prior commit.

As before, I split these into two aggregate queries, hinging on whether
`locked_at` (and therefore, a heap lookup) is required to return a
result.

/no-platform

stack-info: PR: #96, branch: smudge/stack/7
smudge added a commit that referenced this pull request Feb 10, 2026
This folds `max_lock_age` and `max_age` (and by extension,
`age_alert_percent`) into the combined monitor queries introduced in the
prior commit.

As before, I split these into two aggregate queries, hinging on whether
`locked_at` (and therefore, a heap lookup) is required to return a
result.

/no-platform

stack-info: PR: #96, branch: smudge/stack/7
smudge added a commit that referenced this pull request Feb 11, 2026
This folds `max_lock_age` and `max_age` (and by extension,
`age_alert_percent`) into the combined monitor queries introduced in the
prior commit.

As before, I split these into two aggregate queries, hinging on whether
`locked_at` (and therefore, a heap lookup) is required to return a
result.

/no-platform

stack-info: PR: #96, branch: smudge/stack/7
smudge added a commit that referenced this pull request Feb 11, 2026
This folds `max_lock_age` and `max_age` (and by extension,
`age_alert_percent`) into the combined monitor queries introduced in the
prior commit.

As before, I split these into two aggregate queries, hinging on whether
`locked_at` (and therefore, a heap lookup) is required to return a
result.

/no-platform

stack-info: PR: #96, branch: smudge/stack/7
@smudge smudge marked this pull request as draft February 12, 2026 20:14
@smudge smudge changed the base branch from smudge/stack/6 to main February 12, 2026 20:14
@smudge smudge changed the base branch from main to smudge/stack/6 February 12, 2026 20:15
@smudge smudge marked this pull request as ready for review February 12, 2026 20:15
Base automatically changed from smudge/stack/6 to main February 12, 2026 20:32
Copy link
Contributor

@effron effron left a comment

Choose a reason for hiding this comment

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

cool stuff!

domainLGTM

@smudge smudge merged commit d84ad27 into main Feb 12, 2026
47 checks passed
@smudge smudge deleted the smudge/stack/7 branch February 12, 2026 20:46
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.

2 participants