Skip to content

add conservative reconnection strategy; #20

Merged
merged 9 commits into from
Feb 16, 2022

Conversation

phobosxy
Copy link
Contributor

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

@i-vovk i-vovk self-requested a review February 16, 2022 08:04
Copy link
Contributor

@i-vovk i-vovk left a comment

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

include/stream-client/connector/connection_pool.hpp Outdated Show resolved Hide resolved
@i-vovk i-vovk force-pushed the add-reconnection-strategy branch from 854390a to 8a21398 Compare February 16, 2022 08:14
@i-vovk

This comment was marked as resolved.

@i-vovk

This comment was marked as resolved.

@i-vovk i-vovk force-pushed the add-reconnection-strategy branch from 8a21398 to affed82 Compare February 16, 2022 08:44
Copy link
Contributor

@i-vovk i-vovk left a comment

Choose a reason for hiding this comment

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

LGTM

@i-vovk i-vovk merged commit bcd9b46 into Tinkoff:develop Feb 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants