-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[META] Build OpenSearch SDK for extensions #1619
Comments
Would be great to see the evaluation of advantages and downsides of running plugins as independent processes. Would this mean memory commitments in general? Plugins also interact with the core and has listeners/interceptors configured which is cheaper in-process. Is the plan for specific plugins or in-general all plugins |
Thanks @Bukhtawar for your thoughts. Please take a look at #1422. There is a high level proposal listed on the issue. We are trying to achieve plugin sandboxing but started with a POC with independent plugins. If you have any thoughts/suggestions let us know. |
Looks like we are doing a POC at this point, would be great to see few comparisons
|
Absolutely. We are still working on the POC and in this order:
But thanks for the suggestions. |
Is your feature request related to a problem? Please describe.
Currently plugins are tightly coupled with OpenSearch making it complex to independently ship enhancements. Running plugin as a separate process will uncover advantages like dynamic loading of plugins and OpenSearch cluster not getting affected even if plugin fails.
11. Call Overridden functions like getRepositories() -https://github.com/opensearch-project/OpenSearch/blob/main/plugins/repositorys3/src/main/java/org/opensearch/repositories/s3/S3RepositoryPlugin.java#L103
12. Integrate Extension with SDK and Remove pluginsService dependencies from repository-s3 plugin. POC: Integrate SQL plugin with SDK #1695Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: