Increase the default size of FD_SETSIZE

[ThomsonTan]

Fixes #1476.

Changes

Please provide a brief description of the changes here.

For significant contributions please make sure you have completed the following items:

• CHANGELOG.md updated for non-trivial changes
• Unit tests have been added
• Changes in public API reviewed

[codecov]

Codecov Report

Merging #1477 (0548de7) into main (9efb7a1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1477   +/-   ##
=======================================
  Coverage   85.31%   85.31%           
=======================================
  Files         154      154           
  Lines        4377     4377           
=======================================
  Hits         3734     3734           
  Misses        643      643           

Impacted Files	Coverage Δ
...lemetry/ext/http/client/curl/http_operation_curl.h	77.64% <ø> (ø)

[lalitb]

Just wondering if the issue would be still valid once @owent async changes are merged, as that PR removes select call usage. Also, there was similar fix done for Linux - #1410 to replace select call with poll. If there is similar alternative for Windows ?

[ThomsonTan]

Just wondering if the issue would be still valid once @owent async changes are merged, as that PR removes select call usage. Also, there was similar fix done for Linux - #1410 to replace select call with poll. If there is similar alternative for Windows ?

@lalitb thanks for the reminder. Yes, the async change will remove current code with issue, so I think I could close this PR once the async change is merged. Also I think #1410 applies to Windows as well, right?

[lalitb]

Also I think #1410 applies to Windows as well, right?

#1410 replaces select with poll only for POSIX and keeps select for Windows. Merging this PR will add more conflict with async PR. If this is not the blocker, we can let @owent to decide how to proceed with this PR?

[ThomsonTan]

Ok. I'll just leave this PR open and will close it once the async change from @owent gets merged, no need for merging.

[owent]

libcurl use select on WIndows and poll on Linux, just like #1410 . According to https://docs.microsoft.com/en-us/windows/win32/api/winsock2/nf-winsock2-select, nfds of select on Windows is just ignored. I think we can wait for #1413 to be merged. Or we can just ignore max_fd checking on Windows.