Skip to content
New issue

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

Added an option to escape delimiters in CSV files when using bulk copy #1312

Merged
merged 6 commits into from
May 26, 2020

Conversation

ulvii
Copy link
Contributor

@ulvii ulvii commented Apr 14, 2020

Fix for #1173

Adding SQLServerBulkCSVFileRecord.setEscapeColumnDelimitersCSV. When set to true, the following rules will apply.

  • Each field may or may not be enclosed in double quotes.
  • If fields are not enclosed with double quotes, then double quotes may not appear inside the fields.
  • Fields containing double quotes, and delimiters should be enclosed in double quotes.
  • If double-quotes are used to enclose fields, then a double-quote appearing inside a field must be escaped by preceding it with another double quote.

@ulvii ulvii added this to the 8.3.1 milestone Apr 16, 2020
@peterbae peterbae added this to Under Peer Review in MSSQL JDBC May 8, 2020
@ulvii ulvii added the Public API Changes in Public API label May 9, 2020
@ulvii ulvii merged commit 439737c into microsoft:dev May 26, 2020
MSSQL JDBC automation moved this from Under Peer Review to Closed/Merged PRs May 26, 2020
@ulvii ulvii deleted the issue1173 branch May 26, 2020 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Public API Changes in Public API
Projects
MSSQL JDBC
  
Closed/Merged PRs
Development

Successfully merging this pull request may close these issues.

None yet

4 participants