remove precomputed content length header from vertx impls

[awmcc90]

• Removes precomputed content length header
• Cancels:
  • [kotlin/vertx-web-kotlin-dsljson] Mark tests as stripped #10661
  • [java/vertx] Mark tests as stripped #10658
• Testing:

./tfb \
  --mode benchmark \
  --test vertx-web-kotlin-dsljson vertx \
  --type json

[awmcc90]

@vietj for visibility

[vietj]

what is the motivation for this ?

[awmcc90]

^ @p8 and @joanhey

[joanhey]

JSON Serialization
The JSON Serialization test exercises the framework fundamentals including keep-alive support, request routing, request header parsing, object instantiation, JSON serialization, response header generation, and request count throughput.

We need to generate the headers, not hard coded or with constants.

#10611 (comment)

[p8]

@vietj Previous discussions:
#10611 (comment)
#10624 (review)
#9916 (comment)
#8205
#9440
#9578

[joanhey]

Please add the maintainers in the benchmark_config.json.

So with any change or marked as broken, it'll be notified to them.

Example:

FrameworkBenchmarks/frameworks/Java/quarkus/benchmark_config.json

Lines 1 to 6 in 33531b8

	{
	"framework": "quarkus",
	"maintainers": ["franz1981", "Sanne", "geoand"],
	"tests": [
	{
	"default": {

[joanhey]

Please add the maintainers, or the next time that is marked as broken, nobody will be notified.

[awmcc90]

In the future please scope all work upfront to prevent a second pass. Thanks.

[joanhey]

Please, help the other people that work for you.
Thank you !!

[joanhey]

I'll explain how you was notified.
The easy part, it's mark the frameworks as broken or stripped.

I spend my time to notify the devs, for that I need to check in Github the framework and later the devs.

It's OK, but I'll do it one time, not more.

Thank you !!!

[joanhey]

If it's to much work to add the maintainers, I can understate it.
But the next time nobody will be notified.

Thank you.