Feature/overide sniffed csv settings#66
Conversation
| "next_page_url_key", "is_next_page_url_relative", "next_page_url_base", | ||
| "top_key", "skip_key", "maximum_number_rows", | ||
| "use_mtls", "mtls_certificate_path", "mtls_key_path", | ||
| "force_csv_parameters", "csv_delimiter" |
There was a problem hiding this comment.
why "force_csv_parameters", "csv_delimiter" are added only and not others ?
adds these too no ?
"csv_doublequote", "csv_escapechar", "csv_lineterminator", "csv_quotechar", "csv_quoting", "csv_skipinitialspace"
|
@alexbourret strange to display labels on the right for the connector |
| dialect.delimiter, | ||
| dialect.doublequote, | ||
| dialect.escapechar, | ||
| dialect.lineterminator, |
There was a problem hiding this comment.
| dialect.lineterminator.encode(encoding="utf-8"), |
There was a problem hiding this comment.
I'm not sure I understand that one....
There was a problem hiding this comment.
if dialect.lineterminator = "\r\n" the trace logger.info("Decoding CSV method is broken
There is no much I can do on that one though, it's either on the right, or on the top like this: |
top is better IMO |
👍 changed in bba5751 |
|
found a bug in for example: |
|
codex finding
|






https://app.shortcut.com/dataiku/story/308030