gsserge opened a new issue #1328:
URL: https://github.com/apache/arrow-rs/issues/1328


   Discovered in 
https://github.com/apache/arrow-rs/pull/1324#discussion_r808432375
   
   The delimiter can be correctly configured via `WriterBuilder`. 
`WriterBuilder::build()` constructs the `Writer` in-line and does not use 
`Write::new()`. 
   
   Currently, `Write::new()` does not have the ability to configure the 
delimiter. The function should be changed to accept it as a parameter, and then 
`WriterBuilder::build()` should use `Write::new()` instead of duplicating the 
code. The `delimiter` field in `struct Writer` can be removed, because it can 
only be configured when creating `Writer`. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to