Skip to content
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

Minor retention tweaks #3856

Merged
merged 3 commits into from
Aug 27, 2015
Merged

Minor retention tweaks #3856

merged 3 commits into from
Aug 27, 2015

Conversation

otoolep
Copy link
Contributor

@otoolep otoolep commented Aug 26, 2015

Minor changes:

  • fix typo in log message.
  • increase retention check period since the minimum retention period is 1 hour.

@otoolep otoolep added this to the 0.9.4 milestone Aug 26, 2015
@otoolep otoolep self-assigned this Aug 26, 2015
@toddboom
Copy link
Contributor

+1

otoolep added 3 commits August 27, 2015 16:08
@otoolep
Correct typo in retention service logs
ae825fd
@otoolep
Set default retention check interval to 30 minutes
d771612 
Since the minimum retention period is 1 hour, checking every 10 minutes
seems excessive and generates noise in the logs.
@otoolep
Update CHANGELOG
3184103
otoolep added a commit that referenced this pull request Aug 27, 2015
@otoolep
Merge pull request #3856 from influxdb/retention_tweaks
5cf8920 
Minor retention tweaks
@otoolep otoolep merged commit 5cf8920 into master Aug 27, 2015
@otoolep otoolep deleted the retention_tweaks branch August 27, 2015 23:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@otoolep otoolep

Labels
None yet
Projects
None yet
Milestone
0.9.4
Development

Successfully merging this pull request may close these issues.
2 participants
@otoolep @toddboom