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

[Backport 2.x] Cleanup interfaces around ExtensionManager (#7374) #7451

Merged
merged 1 commit into from
May 9, 2023
Merged

[Backport 2.x] Cleanup interfaces around ExtensionManager (#7374) #7451

merged 1 commit into from
May 9, 2023

Commits on May 5, 2023

  1. Cleanup interfaces around ExtensionManager (opensearch-project#7374) 

    * Cleanup interfaces around ExtensionManager

ExtensionsManager has a whole lot of functionality that shouldn't be
exposed to external classes, marked much of this package protected for a
cleaner interface.  Then updated the Noop version to override all public
functions to remove extranious feature flag checks.

Note; I didn't go so far to remove all getters/setters that were unused
but that could be a good follow up in the future.

Signed-off-by: Peter Nied <[email protected]>
(cherry picked from commit 3687d01)
    @peternied
    peternied committed May 5, 2023
    Configuration menu
    Copy the full SHA
    4eb2148 View commit details
    Browse the repository at this point in the history