-
Notifications
You must be signed in to change notification settings - Fork 35
BulkExport.to_str() writes when delimiter is present in table #45
Comments
Checking the delimiter for every byte of data would introduce a very significant performance change to cover this case. While I agree that it would be nice to find that the source data has a character that is being used for a delimiter, but I think for the most part this is why the I think it is possible to better abstract this so that it is either more apparent how to solve issues like in your example, or to just provide a better interface that would solve for this automatically. For example, |
If I have a table with a pipe (
|
) in a character field and try to export and write to a string, there's no checking for whether the delimiter exists in any of the fields:I could bring it out with a different delimiter, or roll my own
to_str
that takes in adict
fromto_dict
/to_list
, but wondering if there should be functionality pushed up intoto_str
to error out, warn, or remove the field if the delimiter exists in a column?The text was updated successfully, but these errors were encountered: