-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Refactor backup & restore #2886
Conversation
1e347b8
to
d82cc88
Compare
fixes #2812 ❓ |
@neonstalwart Yes. Thanks for tagging the issue. |
if path == "" { | ||
return url.URL{}, "", fmt.Errorf("snapshot path required") | ||
// Ensure that only one arg is specified. | ||
if fs.NArg() == 0 { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor -- this could be combined into a single check, if the error message was "a single snapshot path is required". This is purely a style issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I went back and forth on it. I like the extra clarity. I don't have a terribly strong opinion about it though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Keep it your way. All good here.
A lot here, took a look, but don't grok it all. I do have one specific question about a WaitGroup. |
58bd109
to
d3d6d73
Compare
This commit updates the snapshot code as well as the "backup" and "restore" command to work with the new architecture.
d3d6d73
to
bc31783
Compare
Overview
This pull request refactors the
influxd backup
andinfluxd restore
commands to work with the new architecture. This involved adding additional protobuf support to themeta
package to serialize themeta.Data
tree and refactoring snapshots into thesnapshots
package and making them work with the new file layout.