Skip to content

add -csv.delim flag to zed and zq#4325

Merged
nwt merged 4 commits intobrimdata:mainfrom
dianetc:non-comma-seperator-csv
Jan 24, 2023
Merged

add -csv.delim flag to zed and zq#4325
nwt merged 4 commits intobrimdata:mainfrom
dianetc:non-comma-seperator-csv

Conversation

@dianetc
Copy link
Contributor

@dianetc dianetc commented Jan 22, 2023

The -csv.delim flag specifies the field delimiter for CSV input, allowing use of a character other than ",".

For #4238.

@philrz philrz requested a review from nwt January 22, 2023 17:54
@dianetc dianetc changed the title potential solution to allowing non comma seperator for csv potential solution to allowing semicolon seperator for csv Jan 22, 2023
Copy link
Member

@nwt nwt left a comment

Choose a reason for hiding this comment

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

Looking good!

Copy link
Member

@nwt nwt left a comment

Choose a reason for hiding this comment

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

Just one last tweak before we merge this.

@nwt nwt changed the title potential solution to allowing semicolon seperator for csv add -csv.delim flag to zed and zq Jan 24, 2023
@nwt nwt merged commit 4aa010c into brimdata:main Jan 24, 2023
@nwt
Copy link
Member

nwt commented Jan 24, 2023

Thank you, @dianetc!

@philrz philrz linked an issue Jan 24, 2023 that may be closed by this pull request
@dianetc dianetc self-assigned this Apr 17, 2023
@philrz philrz mentioned this pull request Aug 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow non-comma separator for CSV input

2 participants