add conservative reconnection strategy; by phobosxy · Pull Request #20 · Tinkoff/stream-client (original) (raw)

Conversation

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

@phobosxy

the old reconnection strategy is used by default as the default parameter, should not break the build to anyone;

i-vovk

i-vovk

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, rebase on latest develop, there was a mistake and .ipp files weren't linted or format-checked by CI

@i-vovk

This comment was marked as resolved.

@i-vovk

This comment was marked as resolved.

Sivanev Ilya added 6 commits

February 16, 2022 11:44

@i-vovk

old reconnection strytegy is used by default as default parametr, should not break build to anyone;

@i-vovk

@i-vovk

@i-vovk

@i-vovk

@i-vovk

Sivanev Ilya added 3 commits

February 16, 2022 12:51

i-vovk

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

2 participants

@phobosxy @i-vovk