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.

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

Is there any way to limit jersey-server-managed-async-executor thread pool size in helidon mp #1538

Closed
crmouli opened this issue Mar 19, 2020 · 3 comments
Assignees
Labels
bug Something isn't working MP P2

Comments

@crmouli
Copy link

crmouli commented Mar 19, 2020

Need a way to limit jersey-server-managed-async-executor thread pool size in helidon mp

Environment Details

  • Helidon Version:1.4.1
  • Helidon MP
  • JDK version: 1.8
  • OS: ubuntu
  • Docker version (if applicable):

Problem Description

We are using AsyncResponse with helidon-mp and want to see if there is any performance improvement if we use same shared thread pool across the app. So want to know if there is a way to replace this jersey async thread pool.

Steps to reproduce

Start helidon-mp server and make any get request and see the thread count by connecting jvisualvm

@m0mus m0mus added bug Something isn't working MP P2 labels Mar 26, 2020
@tomas-langer tomas-langer self-assigned this Mar 26, 2020
@tomas-langer
Copy link
Member

Related to #1540

@tomas-langer
Copy link
Member

There seems to be one case where Jersey uses its own executor service that is not wrapped by a Helidon one, so we lose track of Context.

@tomas-langer
Copy link
Member

The PR (one for master will follow shortly) adds support to configure the managed async executor service.
You can configure both the default and async executor services through the usual configuration mechanisms, example using META-INF/microprofile-config.properties:

# configure custom prefix for Jersey executor service (server)
server.executor-service.thread-name-prefix=jersey-
# configure custom prefix for Jersey async executor service (server)
server.jersey.async-executor-service.thread-name-prefix=jersey-async

For all configurable options see: https://helidon.io/docs/latest/apidocs/index.html?overview-summary.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working MP P2
Projects
Archived in project
Development

No branches or pull requests

3 participants