Removing Host header from wrk command line

[sebastienros]

Would it make sense to remove the Host header argument form the wrk command line: https://github.com/TechEmpower/FrameworkBenchmarks/blob/master/toolset/wrk/pipeline.sh#L31

Since wrk will include it either way based on the target url. I haven't measured the perf impact on the load server, but at least it would simplify the command line.

[sebastienros]

And Connection: keep-alive is also implicit in HTTP 1.1

[michaelhixson]

Our manually-specified Host header is also wrong because it does not include the port number. Agreed, it makes sense to let wrk set this.

We assume removing Connection: keep-alive would cause an increase in throughput -- it would decrease the size of requests over the network and reduce the header-parsing burden of the server -- resulting in some churn in the results, but internally no one had a problem with this.