Skip to content

Support touch function #48

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 3 commits into from
Closed

Support touch function #48

wants to merge 3 commits into from

Conversation

@aensidhe
Copy link
Contributor

@aensidhe aensidhe commented Dec 16, 2016 •
edited
Loading

Closes #47

This change is Reviewable

@aensidhe
Copy link
Contributor Author

aensidhe commented Dec 26, 2016 •
edited
Loading

TODO:

  • rename extend_ttr to touch
  • if we're extending ttr - extend ttl to same interval
  • if ttr or ttl larger than TIMEOUT_INFINITY, then touch is noop.

@aensidhe aensidhe changed the title Support extend_ttr function Support touch function Dec 30, 2016
bigbes added a commit that referenced this pull request Jan 16, 2017
@bigbes
Review fixes for gh-48. closes gh-47, gh-48
b2016c7
@aensidhe
Copy link
Contributor Author

aensidhe commented Jan 17, 2017

Closed in b2016c7

@aensidhe aensidhe closed this Jan 17, 2017
@aensidhe aensidhe deleted the feature/tasks/extend-ttr branch January 17, 2017 15:53
@rybakit
Copy link
Contributor

rybakit commented Jan 17, 2017 •
edited
Loading

@bigbes @aensidhe What about documenting the new method?
nvm, I see it's already in README

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@aensidhe @rybakit