-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Evaluate how we are exposing the api endpoints #3801
Comments
I like this idea, I think some of your concern can be simplified by providing proper service interaction, for example I don't see special issues with:
Only important things for me to consider for now are performance implications, but at the end of the day we've similar ones, just the code is distributed and not provided only by logstash-core. Worth exploring the idea of having this services, even if for now is just to expose #2611, but with a more general service design. What do you think? |
Yeah, you are correct if we expose different ports which would be in fact a different puma thread pool the problem would be solve if one of the service make a hang. But I think the added complexity to do that doesn't give a lot of values unless we have multiples different inputs requiring a webserver, which isn't the case right now we can always reevaluate that at a later time. |
Agree a lot with you. /purbon On Wed, 9 Dec 2015 15:27 Pier-Hugues Pellerin notifications@github.com
|
We have decided and implement a different service decoupled from the http input server. |
The API will need to expose his endpoints with a webserver and we also have a http input providing his own HTTP server. It's probably a good time to see if we offer the HTTP server as a first class service that plugins authors can hook into and add their own endpoints.
PRO:
CON:
Ref: #2611
The text was updated successfully, but these errors were encountered: