-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Open
Labels
area-System.NetenhancementProduct code improvement that does NOT require public API changes/additionsProduct code improvement that does NOT require public API changes/additionshelp wanted[up-for-grabs] Good issue for external contributors[up-for-grabs] Good issue for external contributorsos-linuxLinux OS (any supported distro)Linux OS (any supported distro)tenet-performancePerformance related issuePerformance related issue
Milestone
Description
Edit by @antonfirsov:
this issue has been reopened to track the missing feature, and the related disabled test (see #70089). We should not use getaddrinfo_a as proposed by the original opening comment, since it turned out to be sub-optimal (see #48666).
We should use getaddrinfo_a on Linux for async name resolution. It supports cancellation as well, bringing us in line with Windows support. See sample code for a usage example.
Note: Today we do async over sync on Linux.
geoffkizer
Metadata
Metadata
Assignees
Labels
area-System.NetenhancementProduct code improvement that does NOT require public API changes/additionsProduct code improvement that does NOT require public API changes/additionshelp wanted[up-for-grabs] Good issue for external contributors[up-for-grabs] Good issue for external contributorsos-linuxLinux OS (any supported distro)Linux OS (any supported distro)tenet-performancePerformance related issuePerformance related issue