Optimize the use of buffers in health reporting

[geoand]

This is done by leveraging Vert.x's Buffer directly instead
of relying on a ByteArrayOutputStream which results in
of unnecessary copying of data

[quarkus-bot]

Failing Jobs - Building 5f67ced

Status	Name	Step	Failures	Logs	Raw logs
✖	Gradle Tests - JDK 11	Build	Failures	Logs	Raw logs
✔️	Gradle Tests - JDK 11 Windows

Full information is available in the Build summary check run.

Failures

⚙️ Gradle Tests - JDK 11 #

- Failing: integration-tests/gradle 

📦 integration-tests/gradle

✖ io.quarkus.gradle.devmode.CompositeBuildWithDependenciesDevModeTest.main line 24 - More details - Source on GitHub

org.awaitility.core.ConditionTimeoutException: Condition with lambda expression in io.quarkus.test.devmode.util.DevModeTestUtils that uses java.util.function.Supplier, java.util.function.Supplierjava.util.concurrent.atomic.AtomicReference, java.util.concurrent.atomic.AtomicReferencejava.lang.String, java.lang.Stringboolean was not fulfilled within 1 minutes.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:78)

[mkouba]

Looks good. I wonder if the BufferOutputStream should go into a util package somewhere?

[geoand]

I thought about that, but I would say let's play it safe and keep it private for now and revise the decision if we need something similar in the future

[mkouba]

I thought about that, but I would say let's play it safe and keep it private for now and revise the decision if we need something similar in the future

+1 expect that nobody would know about this class (and you'll probably forget about it in a few months too ;-).

[geoand]

That is entirely possible 😆