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.

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

move interval to config file from gc/expire work #50

Merged
merged 3 commits into from
Feb 12, 2019

Conversation

nioshield
Copy link
Contributor

Move the gc/expire related configuration to the titan.toml configuration file

@nioshield nioshield added the enhancement New feature or request label Feb 1, 2019
@nioshield nioshield self-assigned this Feb 1, 2019
@nioshield nioshield added this to In progress in Make things stable via automation Feb 1, 2019
Make things stable automation moved this from In progress to Needs review Feb 1, 2019
conf/config.go Outdated Show resolved Hide resolved
db/tikvgc.go Show resolved Hide resolved
@arthurkiller
Copy link
Contributor

LGTM :)

Make things stable automation moved this from Needs review to Reviewer approved Feb 2, 2019
Copy link
Contributor

@YIDWang YIDWang left a comment

Choose a reason for hiding this comment

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

LGTM

Make things stable automation moved this from Reviewer approved to Needs review Feb 2, 2019
conf/config.go Outdated Show resolved Hide resolved
conf/config.go Outdated Show resolved Hide resolved
Copy link
Contributor

@shafreeck shafreeck left a comment

Choose a reason for hiding this comment

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

LGTM

@shafreeck
Copy link
Contributor

@YIDWang Please take another look, your requests have been resolved.

Copy link
Contributor

@YIDWang YIDWang left a comment

Choose a reason for hiding this comment

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

L

Make things stable automation moved this from Needs review to Reviewer approved Feb 12, 2019
@YIDWang YIDWang merged commit 40bca27 into master Feb 12, 2019
Make things stable automation moved this from Reviewer approved to Done Feb 12, 2019
@YIDWang YIDWang deleted the feature/gc-interval-conf branch February 20, 2019 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants