search

janestreet / async_unix

Jane Street Capital's asynchronous execution library (unix)
MIT License
33 stars 21 forks source link

Sequentialized DNS lookup implementation #4

Closed objmagic closed 9 years ago

objmagic commented 9 years ago

At line 237 of unix_syscalls.mli, seems that DNS lookup has been sequentialized due to some thread safety issue.

I talked to Yaron in the IRC and he said

The winbind issue looks like an old one to me. I think we don’t rely on winbind anymore, but I’m not sure. In any case, we can probably make the degree of parallelism tuneable.

Is it possible to make some improvement on it?

Thanks

ghost commented 9 years ago

We discussed it and it turns out we don't need to sequentialize DNS lookups anymore. So this will be removed in the next release.