Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use separate async tracker for SG work instad of reusing workspace async tracker #73880

Merged
merged 1 commit into from
Jun 7, 2024

Conversation

CyrusNajmabadi
Copy link
Member

@CyrusNajmabadi CyrusNajmabadi commented Jun 6, 2024

@dotnet-issue-labeler dotnet-issue-labeler bot added Area-IDE untriaged Issues and PRs which have not yet been triaged by a lead labels Jun 6, 2024
@CyrusNajmabadi CyrusNajmabadi changed the base branch from main to release/dev17.11 June 6, 2024 20:11
@CyrusNajmabadi CyrusNajmabadi changed the base branch from release/dev17.11 to main June 6, 2024 20:11
@CyrusNajmabadi CyrusNajmabadi force-pushed the sgWait branch 2 times, most recently from 42154ab to ddffabf Compare June 6, 2024 20:12
@CyrusNajmabadi CyrusNajmabadi changed the base branch from main to release/dev17.11 June 6, 2024 20:12
…ync tracker

No need to wait to be fully loaded.  all we're doing is incrementing numbers, not actually running generators
@genlu
Copy link
Member

genlu commented Jun 6, 2024

test insertion for main

@CyrusNajmabadi CyrusNajmabadi marked this pull request as ready for review June 7, 2024 06:27
@CyrusNajmabadi CyrusNajmabadi requested a review from a team as a code owner June 7, 2024 06:27
@CyrusNajmabadi CyrusNajmabadi requested a review from genlu June 7, 2024 06:32
@genlu genlu merged commit 072f980 into dotnet:release/dev17.11 Jun 7, 2024
25 checks passed
@CyrusNajmabadi CyrusNajmabadi deleted the sgWait branch June 7, 2024 06:37
@sharwell
Copy link
Member

sharwell commented Jun 7, 2024

If this passes, it indicates a serious bug in the usage of what is now on the SG async tracker. We should definitely fix that. The idea behind the expedited waiter is it forces all time-based waiting items to complete immediately. If this involves items waiting for something other than a timeout, we should find a way to resolve it consistently.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Area-IDE untriaged Issues and PRs which have not yet been triaged by a lead
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants