Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

fix(scheduler-utils): do not cache the redirected url in the byOwner cache #548

Merged
merged 2 commits into from
Mar 18, 2024

Conversation

TillaTheHun0
Copy link
Member

This was the root cause of #543

The url obtained from the redirect, which is process specific, was being cached for the owner. So when calling raw on scheduler-utils, it was returning the cached redirected url for the specific process. This would cause subsequent processes to 404 on the scheduler.

This PR also adds the ability to provide aschedulerHint to locate, which will cause it to skip querying for the process, and instead query directly for the Scheduler-Location, utilizing it's internal in memory cache.

@TillaTheHun0 TillaTheHun0 merged commit 5b26a4d into main Mar 18, 2024
2 checks passed
@TillaTheHun0 TillaTheHun0 deleted the tillathehun0/scheduler-utils-cache branch March 18, 2024 16:55
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.

1 participant