Skip to content

chore: remove async getaddrinfo code as it's no longer used

Georgy Moshkin requested to merge gmoshkin/fix-warnings into master

Summary

  • chore: remove async getaddrinfo code as it's no longer used

We don't use async getaddrinfo in the TcpStream anymore. You shouldn't be creating connections concurrently within a fiber with other yielding operations anyway, it's always possible to rewrite this kind of code and make it simpler doing so.

In principle it's not impossible to make this sort of thing work, but for now it requires getting more flexible access to the libev which underlies the fiber runtime, and for that we already have some work in progress. Should it land in tarantool & tarantool-module, we'll have a simpler way to do async getaddrinfo.

  • chore: fix compile warnings

Ensure that

  • New code is covered by tests
  • API is documented
  • Changelog is up to date
  • Version is bumped in the appropriate Cargo.toml files

Merge request reports