Skip to content
This repository has been archived by the owner on Aug 15, 2018. It is now read-only.

Довести код http-транспорта до одинакового уровня качества под все платформы #41

Open
beevee opened this issue Nov 24, 2017 · 2 comments

Comments

@beevee
Copy link
Contributor

beevee commented Nov 24, 2017

https://github.com/vostok/core/blob/b3754e739dbc488bd31f0614abb5b71653cb9515/Vostok.ClusterClient.Transport.Http/netstandard/VostokHttpTransport_Standard.cs#L13-L14

Эти тудушки точно нужно сделать, и плюс посмотреть, чего еще не хватает по сравнению с Windows-специфичной реализацией.

@trurl123
Copy link
Contributor

trurl123 commented Dec 1, 2017

покурил эту тему, получается вот что (это же отправил Илье):

Tune CurlHandler in case it backs our handler (see SetCurlOption function with CURLOPT_CONNECTTIMEOUT_MS)
CurlHandler - это internal class, многие SetCurlOption не связаны явно с возможностью настройки через публичные методы.
Например, CURLOPT_CONNECTTIMEOUT_MS точно никак не получится поменять, эта настройка всегда устанавливается в int.MaxValue
т.е. тюнить можно только через публичные методы и свойства HttpClient, HttpClientHandler, что уже сейчас делается.
Есть какие-то Properties в HttpClientHandler, но они никак не используются.

Classify errors from CurlHandler - CurlException хоть и бывает иногда в качестве inner exception, но добраться до свойств CurlException не получится, это internal class.
Часто CurlException преобразуется в другие исключения, которые уже есть в ядре.
Ну и какой-то код ошибки из curl я не нашел как получить.

@trurl123
Copy link
Contributor

trurl123 commented Dec 5, 2017

Сейчас пока в этом тикете не видно необходимости. Возможно, если у нас будут адские нагрузки и всё будет падать, то может быть стоит подумать о том, что делать хаки. Сейчас используются стандартные средства .net, они уже протестированы микрософтом. Т.е. тикет предполагает то, что микрософт что-то сделали неоптимально и нам надо тюнить. Чтобы понять, что делать, необходимо провести довольно большое нагрузочное тестирование на множестве серверов.

@trurl123 trurl123 removed their assignment Dec 15, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants