Skip to content

Update dependencies in ring-http-exchange#10765

Merged
msmith-techempower merged 1 commit intoTechEmpower:masterfrom
ruroru:master
Feb 17, 2026
Merged

Update dependencies in ring-http-exchange#10765
msmith-techempower merged 1 commit intoTechEmpower:masterfrom
ruroru:master

Conversation

@ruroru
Copy link
Contributor

@ruroru ruroru commented Feb 16, 2026

Also updated executor and hikaricp connection settings

Also updated executor and hikaricp connection settings
@joanhey
Copy link
Contributor

joanhey commented Feb 16, 2026

Please add yourself to the maintainers in the benchmark_config.json.

So with any change or marked as broken, you will be notified.

Example:

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

Comment on lines +19 to +20
"/plaintext" (Response. model/hello-world 200 plain-text-headers)
"/json" (Response. (model/json-body) 200 json-headers)
Copy link
Contributor

@joanhey joanhey Feb 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, do you need the model for send a plaintext/json response without database ??
It's only a comment !!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see a problem with this. Looks like it's properly serializing the json on every request.

Comment on lines +19 to +20
"/plaintext" (Response. model/hello-world 200 plain-text-headers)
"/json" (Response. (model/json-body) 200 json-headers)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see a problem with this. Looks like it's properly serializing the json on every request.

@msmith-techempower msmith-techempower merged commit 5a1d936 into TechEmpower:master Feb 17, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants