Skip to content

Conversation

@kosak
Copy link
Contributor

@kosak kosak commented Jan 15, 2026

NOTE TO REVIEWERS: This PR is downstream of #308. It will be easier to read this PR once that one is merged.

Add opt-in support for an escape character. This allows two features:

  1. Support for C-style escape characters (\b \t \n \r \f)
  2. A way to escape the quote character or the field separator (e.g. ,)

For example, if \ is enabled as the escape character in CsvSpecs, then this input will be parsed with an embedded newline

hello\nthere

Likewise, this input will be parsed as two cells: "hello, world" and "42"
hello\, world, 42

@kosak kosak requested a review from lbooker42 January 15, 2026 05:11
@kosak kosak self-assigned this Jan 15, 2026
@kosak kosak force-pushed the kosak_escape-character branch 2 times, most recently from 3a4e124 to b29d8d4 Compare January 15, 2026 05:33
@kosak kosak force-pushed the kosak_escape-character branch 5 times, most recently from 4122122 to 2210d2f Compare January 15, 2026 18:03
@kosak kosak force-pushed the kosak_escape-character branch from 2210d2f to 7a66d24 Compare January 15, 2026 18:12
@kosak kosak merged commit 1f6c654 into deephaven:main Jan 15, 2026
3 checks passed
@kosak kosak deleted the kosak_escape-character branch January 15, 2026 22:38
@github-actions github-actions bot locked and limited conversation to collaborators Jan 15, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants