Am 2019-10-11 um 11:51 schrieb Rémy Maucherat:
On Fri, Oct 11, 2019 at 10:30 AM <> wrote:

This is an automated email from the ASF dual-hosted git repository.

michaelo pushed a commit to branch BZ-63835/8.5.x
in repository

commit 6ff2233cbbd27c9c2c649208a21931e5f3e132a6
Author: Michael Osipov <>
AuthorDate: Fri Oct 11 10:30:08 2019 +0200

     First draft

+                if (keepAliveTimeout > 0) {
+                    String value = "timeout=" +
+                    if (maxKeepAliveRequests > 0) {
+                        value += ", max=" + maxKeepAliveRequests;
+                    }
StringBuilder ?

StringBuilder does not make any sense because:

* The compiler will replace the static code automatically with a StringBuilder * StringBuilder gains benefit when you concat strings in a for/while/do loop. The above code is purely static.

Can you add a new flag controlling the feature ? The information may not be
very useful in many cases such as when proxying, so it would be better to
skip generating it by default.

This is -- as documented -- a first draft.

As mentioned on the ticket. This is hop-by-hop and writetn only if the client requests this piece of information. We can surely discuss a flag for this on the connector.

To unsubscribe, e-mail:
For additional commands, e-mail:

Reply via email to