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

The default value of FD_SETSIZE is too small for real scenario #1476

Closed
ThomsonTan opened this issue Jul 5, 2022 · 2 comments
Closed

The default value of FD_SETSIZE is too small for real scenario #1476

ThomsonTan opened this issue Jul 5, 2022 · 2 comments
Assignees
@ThomsonTan
Labels
bug Something isn't working

Comments

@ThomsonTan
Copy link
Contributor

Describe your environment
Windows 11 10.0.22000.739

Steps to reproduce
Run curl_http_test.exe

What is the expected behavior?
The test should pass

What is the actual behavior?
The test failed

Additional context
The test is failed on that the allocated socket handle value is larger than FD_SETSIZE which is 64. The value seems too small for real scenario.
@ThomsonTan ThomsonTan added the bug Something isn't working label Jul 5, 2022
@ThomsonTan ThomsonTan self-assigned this Jul 5, 2022
@ThomsonTan ThomsonTan mentioned this issue Jul 5, 2022
Closed
3 tasks
@owent
Copy link
Member

owent commented Jul 9, 2022

#1413 is already merged, maybe this issue can also be closed?

@ThomsonTan
Copy link
Contributor Author

Resolved by #1413.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ThomsonTan ThomsonTan

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Increase the default size of FD_SETSIZE ThomsonTan/opentelemetry-cpp
2 participants
@ThomsonTan @owent