Skip to content

Refactor connector cleanup#357

Merged
asvetlov merged 2 commits intomasterfrom
refactor_connector_cleanup
May 11, 2015
Merged

Refactor connector cleanup#357
asvetlov merged 2 commits intomasterfrom
refactor_connector_cleanup

Conversation

@asvetlov
Copy link
Copy Markdown
Member

@asvetlov asvetlov commented May 11, 2015

  1. Make 1-second grained step interval (e.g call cleanup procedure on second boundaries, not fractions).
  2. Calculate time to next cleanup as minimum required one, not self._keepalive_timeout constant.

@fafhrd91
Copy link
Copy Markdown
Member

fafhrd91 commented May 11, 2015

lgtm

asvetlov added a commit that referenced this pull request May 11, 2015
@asvetlov
Merge pull request #357 from KeepSafe/refactor_connector_cleanup
f99348b 
Refactor connector cleanup
@asvetlov asvetlov merged commit f99348b into master May 11, 2015
@asvetlov asvetlov deleted the refactor_connector_cleanup branch May 11, 2015 18:42
@lock
Copy link
Copy Markdown

lock bot commented Oct 30, 2019

This thread has been automatically locked since there has not been
any recent activity after it was closed. Please open a new issue for
related bugs.

If you feel like there's important points made in this discussion,
please include those exceprts into that new issue.

@lock lock bot added the outdated label Oct 30, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

outdated

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@asvetlov @fafhrd91