Skip to content

jobs: add a reference to the resumer in the claimed jobs map#109826

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
adityamaru:add-resumer
Sep 1, 2023
Merged

jobs: add a reference to the resumer in the claimed jobs map#109826
craig[bot] merged 1 commit intocockroachdb:masterfrom
adityamaru:add-resumer

Conversation

@adityamaru
Copy link
Copy Markdown
Contributor

This change adds a reference to the resumer in the map maintained by every registry of the jobs that it has claimed. This will be used in a follow up commit to request the resumer to dump its job profiler details on demand as explained in

Informs: #109671
Release note: None

@adityamaru adityamaru requested review from dt and stevendanna August 31, 2023 17:02
@adityamaru adityamaru requested review from a team as code owners August 31, 2023 17:02
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable


ctx := context.Background()
intervalOverride := time.Millisecond
s, _, _ := serverutils.StartServer(t, base.TestServerArgs{
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

😭

This change adds a reference to the resumer in the map
maintained by every registry of the jobs that it has claimed.
This will be used in a follow up commit to request the resumer
to dump its job profiler details on demand as explained in

Informs: cockroachdb#109671
Release note: None
@adityamaru
Copy link
Copy Markdown
Contributor Author

@msbutler is investigating the slow quiesce but its unrelated to this PR so merging this.

bors r=dt

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Sep 1, 2023

Build succeeded:

@craig craig bot merged commit bdf2b4e into cockroachdb:master Sep 1, 2023
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