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.

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

Additional threshold for deleting task history row #1369

Merged
merged 7 commits into from Dec 14, 2016

Conversation

Projects
None yet
1 participant
@ssalinas
Member

ssalinas commented Dec 13, 2016

Building off of the previous history purge update PR and I'll merge the two. This separates out settings for deleting task history bytes versus the entire task history row in the db. Rows to delete are checked/purged first, followed by rows to update bytes.

deleteTaskHistoryBytes... settings take on the previous default values of 365 days/10k tasks since that was the previous default behavior
deleteTaskHistory... settings default to 0 so the rows are not deleted by default

@ssalinas ssalinas added the hs_staging label Dec 13, 2016

@ssalinas ssalinas modified the milestone: 0.14.0 Dec 13, 2016

@ssalinas ssalinas added the hs_qa label Dec 14, 2016

ssalinas added some commits Dec 14, 2016

@ssalinas

This comment has been minimized.

Show comment
Hide comment
@ssalinas

ssalinas Dec 14, 2016

Member

This PR is into the task_history_purge_updates branch, not master. Going to merge it in there now that both are in hs_qa

Member

ssalinas commented Dec 14, 2016

This PR is into the task_history_purge_updates branch, not master. Going to merge it in there now that both are in hs_qa

@ssalinas ssalinas merged commit c1a4f22 into task_history_purge_updates Dec 14, 2016

0 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details

@ssalinas ssalinas deleted the history_purge_thresholds branch Dec 14, 2016

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