davsclaus commented on code in PR #28323:
URL: https://github.com/apache/camel/pull/28323#discussion_r4357368772
##########
components/camel-jetty-common/src/main/java/org/apache/camel/component/jetty/JettyHttpComponent.java:
##########
@@ -137,12 +136,19 @@ static class ConnectorRef {
final Server server;
final ServerConnector connector;
final CamelServlet servlet;
+ // the thread pool created for this server from minThreads /
maxThreads, stopped with the server
+ final QueuedThreadPool threadPool;
int refCount;
ConnectorRef(Server server, ServerConnector connector, CamelServlet
servlet) {
+ this(server, connector, servlet, null);
+ }
+
Review Comment:
Nit: after this change, `connect()` is the only place that creates a
`ConnectorRef`, and it uses the new 4-arg constructor. The 3-arg constructor
has no callers left (`ConnectorRef` is package-private, and nothing in
`camel-jetty`/`camel-jetty-common` uses it). It could be removed:
```suggestion
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
