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.

Sign up for GitHub

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

Jump to bottom

⚠️ Remove apiutil.NewDiscoveryRESTMapper, use DynamicRESTMapper by default for cache #2611

Merged
merged 1 commit into from Dec 6, 2023
Merged

⚠️ Remove apiutil.NewDiscoveryRESTMapper, use DynamicRESTMapper by default for cache #2611

merged 1 commit into from Dec 6, 2023

Commits on Dec 6, 2023

  1. ⚠️ Drop DiscoveryRESTMapper 

    The discovery restmapper was unused except for two places that we missed
(when creating a cache directly as opposed through the manager and in a
test). The discovery restmapper only ever loads mappings once during
startup and is unable to reload them if they change. It also isn't lazy
and we don't want anyone using it.
    @alvaroaleman
    alvaroaleman committed Dec 6, 2023
    Configuration menu
    Copy the full SHA
    cadc020 View commit details
    Browse the repository at this point in the history