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

Provide -lponly option for influx_inspect export & generate only line protocol #19367

Closed
timhallinflux opened this issue Aug 18, 2020 · 6 comments · Fixed by #19609
Closed

Provide -lponly option for influx_inspect export & generate only line protocol #19367

timhallinflux opened this issue Aug 18, 2020 · 6 comments · Fixed by #19609
Assignees
@ayang64
Labels
1.x area/cli
Milestone
1.8.3

Comments

@timhallinflux
Copy link
Contributor

This would allow for for line protocol exports to more easily move between 1.x and 2.x.

Currently, the user needs to crack the export and remove the DML lines.
@psteinbachs psteinbachs modified the milestone: 1.8.3 Aug 18, 2020
ayang64 added a commit that referenced this issue Aug 19, 2020
@ayang64
feat: Add -nodml flag to influx_inspect export
7409fb8 
The nodml flag instructions influx_inspect to skip outputting DML data
when exporting data.

Fixes: #19367
@ayang64 ayang64 mentioned this issue Aug 19, 2020
Closed
4 tasks
ayang64 added a commit that referenced this issue Sep 2, 2020
@ayang64
feat: Add -nodml flag to influx_inspect export
b617ff9 
The nodml flag instructions influx_inspect to skip outputting DML data
when exporting data.

Fixes: #19367
@dgnorton
Copy link
Contributor

dgnorton commented Sep 21, 2020 •
edited
Loading

@timhallinflux should this be -noddl? DDL creates the database. DML is the data/line-proto. Or, is the intent of this change just to remove all lines that are not line protocol?

@timhallinflux
Copy link
Contributor Author

yes... remove all lines that are not LP. 2.0 doesn't accept the various non-LP stuff at the top.

@ayang64
Copy link
Contributor

ayang64 commented Sep 21, 2020

oh -- so it is the other way around? -noddl?

ayang64 added a commit that referenced this issue Sep 21, 2020
@ayang64
feat: Add -noddl and -nodml flags to influx_inspect export
04214f0 
Likewise, the noddl flag instructions influx_inspect to skip outputting
DDL data when exporting data.

The nodml flag instructions influx_inspect to skip outputting DML data
when exporting data.

Fixes: #19367
@dgnorton
Copy link
Contributor

@ayang64 it means getting rid of any line that isn't line protocol. In other words, remove all the lines starting with # and also the CREATE DATABASE lines.

@timhallinflux since this command line option is removing all of the DDL and part of the DML, leaving only line protocol, should the flag be -dataonly or -rawdata maybe?

@timhallinflux
Copy link
Contributor Author

-lponly ?

@ayang64 ayang64 mentioned this issue Sep 22, 2020
Merged
3 tasks
@ayang64 ayang64 linked a pull request Sep 22, 2020 that will close this issue
feat: Add -lponly flag to export sub-command #19609
Merged
3 tasks
@ayang64 ayang64 changed the title Provide -nodml option for influx_inspect export Provide -lponly option for influx_inspect export & generate only line protocol Sep 22, 2020
@dgnorton
Copy link
Contributor

Implemented in:
master-1.x - PR #19609
1.8.3 - PR #19611

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ayang64 ayang64

Labels
1.x area/cli
Projects
None yet
Milestone
1.8.3
Development

Successfully merging a pull request may close this issue.

feat: Add -lponly flag to export sub-command influxdata/influxdb
4 participants
@dgnorton @ayang64 @psteinbachs @timhallinflux