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

ISPN-10040 Embedded and server thread pool defaults should be the same #6816

Merged

Conversation

danberindei
Copy link
Member

https://issues.jboss.org/browse/ISPN-10040

  • Server thread pools now read from embedded

@tristantarrant
Copy link
Member

[2019-03-29T08:05:12.636Z] [ERROR] /home/infinispan/workspace/Infinispan_PR-6816/server/integration/infinispan/src/main/java/org/jboss/as/clustering/infinispan/subsystem/ThreadPoolResource.java:39:8: Unused import - org.infinispan.factories.KnownComponentNames. [UnusedImports]

@danberindei
Copy link
Member Author

Updated. Sorry, I thought IntelliJ would remove the import when I added the static import for the new method :)

Copy link
Contributor

@karesti karesti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just a minor suggestion

@danberindei
Copy link
Member Author

@karesti updated to introduce a constant for the keepalive timeout in BlockingThreadPoolExecutorFactory

@karesti karesti merged commit baf5462 into infinispan:master Apr 2, 2019
@danberindei danberindei deleted the ISPN-10040_merge_thread_pool branch April 2, 2019 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants