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

Fix legacy restore bug #10072 #10206

Merged
merged 2 commits into from
Feb 11, 2019
Merged

Fix legacy restore bug #10072 #10206

merged 2 commits into from
Feb 11, 2019

Conversation

rda0
Copy link

@rda0 rda0 commented Aug 17, 2018

Required for all non-trivial PRs
  • [x ] Sign CLA (if not already signed)

Fix #10072 see comment

  • Restore original functionality (as documented)
  • Print correct error messages

@ghost ghost added the proposed label Aug 17, 2018
@timhallinflux timhallinflux requested a review from rbetts December 19, 2018 07:07
@mark-rushakoff mark-rushakoff changed the base branch from master to 1.8 January 11, 2019 19:03
@e-dard e-dard added the community-pr Pull requests from the community. Thank you! label Feb 1, 2019
@e-dard
Copy link
Contributor

e-dard commented Feb 1, 2019

@aanthony1243 can you take a look at this? It looks reasonable to me.

@e-dard e-dard requested a review from aanthony1243 February 1, 2019 15:48
@e-dard e-dard merged commit 4226733 into influxdata:1.8 Feb 11, 2019
@rda0 rda0 mentioned this pull request Jul 26, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aanthony1243 aanthony1243 aanthony1243 approved these changes

@rbetts rbetts Awaiting requested review from rbetts

Assignees
No one assigned
Labels
community-pr Pull requests from the community. Thank you! proposed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rda0 @e-dard @aanthony1243