Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Two fixes to C# scheduling #2390

Merged
merged 2 commits into from
Sep 14, 2022
Merged

Two fixes to C# scheduling #2390

merged 2 commits into from
Sep 14, 2022

Conversation

Porges
Copy link
Member

@Porges Porges commented Sep 13, 2022

Two fixes to scheduling code:

  • GetPool was not correct for the VM case (this code is possibly legacy and not used any more)
  • BuildWorkUnit could fetch the same pool multiple times and then fail due to BucketConfig mismatch (on TimeStamp)
    • add a cache to the loop so that we only fetch each pool once

@Porges Porges force-pushed the fix-bucketconfig-comparison branch from 1a02882 to 8c65c60 Compare September 14, 2022 01:58
@Porges Porges enabled auto-merge (squash) September 14, 2022 02:01
@Porges Porges disabled auto-merge September 14, 2022 02:01
@Porges Porges enabled auto-merge (squash) September 14, 2022 02:02
@Porges Porges merged commit f375ee7 into main Sep 14, 2022
@Porges Porges deleted the fix-bucketconfig-comparison branch September 14, 2022 02:11
@ghost ghost locked as resolved and limited conversation to collaborators Oct 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants