We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
csv::WriterBuilder
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
While implementing CSV writing in DataFusion (see https://github.com/apache/arrow-datafusion/pull/7390/files), we would like to be able to check the value of has_headers before actually constructing a csv writer. However, the current API has no way to do read the current values (only modify them): https://docs.rs/arrow-csv/45.0.0/arrow_csv/writer/struct.WriterBuilder.html
has_headers
Describe the solution you'd like It would be nice to have read only access to the fields. Maybe something like
let builder = WriterBuilder::new().has_headers(false); let has_headers = builder.get_has_headers()
It is somewhat unfortunate that the builder already uses has_headers to set the field names rather than with_has_headers
with_has_headers
Describe alternatives you've considered We can keep a copy of has_headers around as @devinjdangelo has done in https://github.com/apache/arrow-datafusion/pull/7390/files
Additional context
The text was updated successfully, but these errors were encountered:
Looks it is easy.
Sorry, something went wrong.
Cleanup CSV WriterBuilder (apache#4735)
d377b64
Cleanup CSV WriterBuilder, Default to AutoSI Second Precision (#4735) (…
556c5ff
…#4909) * Cleanup CSV WriterBuilder (#4735) * Update test * Review feedback * Clippy
label_issue.py automatically added labels {'arrow'} from #4909
label_issue.py
Successfully merging a pull request may close this issue.
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
While implementing CSV writing in DataFusion (see https://github.com/apache/arrow-datafusion/pull/7390/files), we would like to be able to check the value of
has_headers
before actually constructing a csv writer. However, the current API has no way to do read the current values (only modify them): https://docs.rs/arrow-csv/45.0.0/arrow_csv/writer/struct.WriterBuilder.htmlDescribe the solution you'd like
It would be nice to have read only access to the fields. Maybe something like
It is somewhat unfortunate that the builder already uses
has_headers
to set the field names rather thanwith_has_headers
Describe alternatives you've considered
We can keep a copy of has_headers around as @devinjdangelo has done in https://github.com/apache/arrow-datafusion/pull/7390/files
Additional context
The text was updated successfully, but these errors were encountered: