vertx: the seven profiles it was missing - #1249
Open
MDA2AV wants to merge 2 commits into
Open
Conversation
static, static-tls, json-tls, async-db, crud, api-4 and api-16. The entry carried four endpoints; api-4 and api-16 rotate over /baseline11, /json and /async-db, so wiring Postgres brings all three profiles with it. json-tls and static-tls listen on 8081. The verticle is already deployed once per core, so having each instance bind both ports spreads the TLS listener across every event loop rather than parking it on one - the failure #1245 fixed on quarkus. ALPN is off: those two profiles want HTTP/1.1 and no h2 offered. Postgres goes through the reactive pg client, one pool per verticle instance so the harness's connection budget is split across them. postgres:18 negotiates SCRAM-SHA-256, which the client leaves to com.ongres.scram. tags is a JSONB column, so the client hands back a JsonArray rather than text - reading it as a String threw and left async-db and crud answering with an empty body and no content type. Static file bodies are read from disk on every request; sendFile hands the descriptor to the kernel and the pre-compressed sibling is picked per request. crud runs cache-aside on Redis through the vertx redis client, with a 200ms TTL and an explicit delete on update. Validation: 58 passed, 0 failed.
Owner
Author
|
/benchmark --save |
Contributor
|
👋 Benchmark request received. A collaborator will review and approve the run. |
Contributor
Benchmark ResultsFramework:
Full log |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
static,static-tls,json-tls,async-db,crud,api-4andapi-16.Tests go from 6 to 13.
The entry carried four endpoints —
/pipeline,/baseline11,/json/:count,/upload.api-4andapi-16rotate over/baseline11,/jsonand/async-db, so wiring Postgres brings all three profiles with it.json-tls / static-tls
The verticle is already deployed once per core, so each instance binds both ports
and the TLS listener is spread across every event loop rather than parked on one
— the failure #1245 fixed on quarkus. ALPN is off: those two profiles want
HTTP/1.1 negotiated and no h2 offered.
Postgres
The reactive pg client, one pool per verticle instance so the harness's
connection budget is split across them rather than opened by each.
postgres:18negotiates SCRAM-SHA-256, which the client leaves to
com.ongres.scram.Worth recording:
tagsis a JSONB column, so the client hands back aJsonArrayrather than text. Reading it as a String threw inside the successhandler, which left
async-dbandcrudanswering with an empty body and nocontent type — the first validation run failed all eight of their checks that
way while
fortunesstill rendered 202 rows, since it only readsidandmessage.static
Bodies are read from disk on every request —
sendFilehands the descriptor tothe kernel rather than holding the bytes — and the pre-compressed
.br/.gzsibling is picked per request, also from disk.
crud
Cache-aside on the vertx redis client, 200ms TTL, explicit delete on update.
Validation
58 passed, 0 failed.
fortunesis deliberately not included.🤖 Generated with Claude Code