Adar Dembo has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/14440 )

Change subject: webserver: enable HTTP keep-alive
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/14440/3/src/kudu/server/webserver.cc
File src/kudu/server/webserver.cc:

http://gerrit.cloudera.org:8080/#/c/14440/3/src/kudu/server/webserver.cc@509
PS3, Line 509:     if (it == path_handlers_.end()) {
> Btw, you may find that you need to fix this eventually - it was fixed in Im
We talked about this offline. Sounds like you found that Python's urllib ends 
up emitting this, and we have some important clients that use urllib.

I'll fix it.



--
To view, visit http://gerrit.cloudera.org:8080/14440
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic08ef5a268fdf6dea6a8c428b4ab8dac27418dd6
Gerrit-Change-Number: 14440
Gerrit-PatchSet: 4
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]>
Gerrit-Comment-Date: Wed, 16 Oct 2019 23:18:48 +0000
Gerrit-HasComments: Yes

Reply via email to