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.

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

multi: use larger dns hash table for multi interface #9376

Closed
wants to merge 2 commits into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Aug 27, 2022

Have curl_multi_init() use a much larger DNS hash table than used for the easy interface to scale and perform better when used with many host names.

curl_share_init() sets an in-between size.

Inspired-by: Ivan Tsybulin
See #9340

Have curl_multi_init() use a much larger DNS hash table than used for
the easy interface to scale and perform better when used with _many_
host names.

curl_share_init() sets an in-between size.

Inspired-by: Ivan Tsybulin
See #9340
@bagder
Copy link
Member Author

bagder commented Aug 27, 2022

@uranix, what do you think about this approach?

@uranix
Copy link

uranix commented Aug 27, 2022

It is great! 71 is probably fine for our environment and if not could be tuned in compile-time.

@bagder bagder closed this in 7632c0d Aug 28, 2022
@bagder bagder deleted the bagder/multi-handle-dns-hash branch August 28, 2022 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

2 participants