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

Feature request: influx inspect -export should dump WAL files #7115

Closed
mark-rushakoff opened this issue Aug 4, 2016 · 4 comments
Closed
Labels
Milestone

Comments

@mark-rushakoff
Copy link
Contributor

Feature Request

influx inspect -export currently does not read or export data from WAL files. It only reads .tsm files. In its current form, it is unable to extract data-at-rest that has not yet been snapshotted to TSM, which means not all of the data is recoverable with the influx inspect tool.

@jwilder
Copy link
Contributor

jwilder commented Aug 4, 2016

These would need to be appended after the TSM files so that replaying them inserts them in the correct order.

@jwilder
Copy link
Contributor

jwilder commented Aug 11, 2016

I have branch started here: https://github.com/influxdata/influxdb/tree/jw-tsm-wal-inspect

@jwilder jwilder added this to the 1.1.0 milestone Aug 11, 2016
@corylanou
Copy link
Contributor

The first question we need to answer is should a shard export include the wal file?

If not, do we add an option like -include-wal when exporting (or leave it as a separate command like this commit does: 6e60cbb), or have both options?

Additionally, do we want the ability to export an entire directory (retention policy) as well? The functionality for this has been mostly completed already in 2ccf4bd

//cc @dgnorton @jwilder

@jwilder
Copy link
Contributor

jwilder commented Sep 13, 2016

Yes. Export of a shard should include the wal segments automatically as they are part of the data stored in the shard.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants