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

Dump TSM files to line protocol #6522

Merged
merged 4 commits into from
May 3, 2016
Merged

Dump TSM files to line protocol #6522

merged 4 commits into from
May 3, 2016

Conversation

toddboom
Copy link
Contributor

This extends the influx_inspect tool to provide a mechanism for dumping TSM files into line protocol. Probably still needs some usability love (being able to specify an output directory, for example), but is functional as-is.

Sample output:

$ go install ./... && $GOPATH/bin/influx_inspect dump
Opening /Users/todd/.influxdb/data/_internal/monitor/2/000000001-000000001.tsm in database _internal
Writing 75 series to /tmp/influxdb183681029/_internal.lp
Opening /Users/todd/.influxdb/data/stress/default/1/000000004-000000003.tsm in database stress
Writing 100000 series to /tmp/influxdb179802528/stress.lp
Opening /Users/todd/.influxdb/data/stress/default/3/000000001-000000001.tsm in database stress
Writing 2 series to /tmp/influxdb653771391/stress.lp
  • Rebased/mergable
  • Tests pass
  • CHANGELOG.md updated
  • Sign CLA (if not already signed)

@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @joelegasse to be a potential reviewer

jwilder
jwilder reviewed Apr 30, 2016
View reviewed changes
cmd/influx_inspect/main.go
@@ -129,7 +129,24 @@ func main() {
fmt.Printf("%v", err)
os.Exit(1)
}
cmdVerify(path)
cmdDump(path)
case "dump":
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe call this export?

@jwilder jwilder added this to the 0.13.0 milestone Apr 30, 2016
@toddboom
Copy link
Contributor Author

toddboom commented May 2, 2016

@jwilder good catches. i'll get those fixed and added today.

@toddboom
Copy link
Contributor Author

toddboom commented May 3, 2016

@jwilder ok, addressed the feedback. let me know if this looks good and i'll squash and merge.

@toddboom toddboom changed the title [WIP] Dump TSM files to line protocol Dump TSM files to line protocol May 3, 2016
@jwilder
Copy link
Contributor

jwilder commented May 3, 2016

Needs a changelog. But 👍

@jwilder jwilder merged commit 57cb3fd into master May 3, 2016
@jwilder jwilder deleted the tp-tsm-dump branch May 3, 2016 16:44
@kfitzpatrick
Copy link
Contributor

Where are the associated docs for this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.13.0
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@toddboom @mention-bot @jwilder @kfitzpatrick