Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add the ability to configure vertx http client options in RESTEasy Reactive client #15189

Merged
merged 1 commit into from
Feb 19, 2021
Merged

Add the ability to configure vertx http client options in RESTEasy Reactive client #15189

merged 1 commit into from
Feb 19, 2021

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Feb 19, 2021 •
edited
Loading

Allows us to set whatever Vert.x HTTP Client options we like by doing do something like:

new ClientBuilderImpl().httpClientOptions(new HttpClientOptions().setMaxPoolSize(100)).build()

instead of

ClientBuilder.newClient()

@geoand geoand requested a review from FroMage February 19, 2021 12:01
@geoand geoand added triage/backport? triage/waiting-for-ci Ready to merge when CI successfully finishes labels Feb 19, 2021
@FroMage FroMage mentioned this pull request Feb 19, 2021
Open
@geoand geoand merged commit cac7740 into quarkusio:master Feb 19, 2021
@quarkus-bot quarkus-bot bot added this to the 1.13 - master milestone Feb 19, 2021
@geoand geoand deleted the jaxrs-client-httpoptions branch February 19, 2021 17:08
@gsmet gsmet modified the milestones: 1.13 - master, 1.12.1.Final Feb 19, 2021
@dependabot dependabot bot mentioned this pull request Mar 11, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FroMage FroMage FroMage approved these changes

@cescoffier cescoffier cescoffier approved these changes

Assignees
No one assigned
Labels
triage/waiting-for-ci Ready to merge when CI successfully finishes
Projects
None yet
Milestone
1.12.1.Final
Development

Successfully merging this pull request may close these issues.
4 participants
@geoand @FroMage @cescoffier @gsmet