Daniel (dB.) Doubrovkine
Daniel (dB.) Doubrovkine
@fddattal I like this. Do you think it makes sense to reuse https://github.com/opensearch-project/opensearch-sdk-java for that and possibly produce multiple artifacts (that eventually merge)?
@owaiskazi19 whoa I totally forgot about that one! full circle
> With `opensearch-java-sdk-api` and `o.o.sdk.Client` will you essentially be implementing a "version 2" of this Plugin interface that doesn't expose any `:server` dependencies? Are there additional complexities here that I'm...
> I agree with this. Probably something like a new library in `libs/pluginv2` or `libs/plugin-api` (or whatever we want to name it). I think this creates the groundwork for integration...
> I think revamping / extending the plugin APIs (`org.opensearch.plugins.Pluginv2`) is the step in reverse direction, that would make the extensibility even more difficult to evolve and maintain. Extensions =...
@reta First, as always, thank you. I really appreciate, value, and respect your strong opinions. One of the things @fddattal says above is "I want to run the SQL plugin...
@dependabot recreate
Late to this game coming from another PR. I think the name "offline" is confusing, would call these "worker" nodes.
Added onError callback (currently plugin eats errors, such as 404). Sorry for two features here, let me know if one of those is a no go and I'll figure something...
Btw, @envygeeks @Bartuz I didn't mean to start too many debates :) I just wanted the person who made a PR, #343 to catch the style issue they were asked...