>From Ian Maxon <[email protected]>:
Attention is currently required from: Ian Maxon.
Hello Jenkins, Murtadha Hubail,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21026?usp=email
to look at the new patch set (#3).
Change subject: [ASTERIXDB-2877][EXT] Fix multi-byte/emoji character corruption
in CSV output
......................................................................
[ASTERIXDB-2877][EXT] Fix multi-byte/emoji character corruption in CSV output
- user model changes: no
- storage format changes: no
- interface changes: no
This is a version of this change for lumina which:
1. Doesn't attempt to restrict the domain of escapes, delimiter and
quote chars to ASCII
2. Doesn't remove the use of PrintStream::print which creates a new
String for every invocation, and can potentially encode into
something other than UTF-8
Ext-ref: MB-71405
Co-authored-by: Ian Maxon<[email protected]>
Generated-by: Gemini 3.1 Pro Preview
(Emoji list in multibyte-chars-02.update.sqlpp)
Change-Id: I434142a9b9cd2d1fc941b1e1f350e97403a8a3e2
---
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/copy-to/csv/multibyte-chars/multibyte-chars.01.ddl.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/copy-to/csv/multibyte-chars/multibyte-chars.02.update.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/copy-to/csv/multibyte-chars/multibyte-chars.03.ddl.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/copy-to/csv/multibyte-chars/multibyte-chars.04.update.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/copy-to/csv/multibyte-chars/multibyte-chars.05.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/copy-to/csv/multibyte-chars/multibyte-chars.99.ddl.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/results/copy-to/csv/multibyte-chars/multibyte-chars.01.adm
M
asterixdb/asterix-app/src/test/resources/runtimets/testsuite_external_dataset_s3.xml
M
asterixdb/asterix-cloud/src/main/java/org/apache/asterix/cloud/CloudResettableInputStream.java
M
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/printers/PrintTools.java
A
asterixdb/asterix-om/src/test/java/org/apache/asterix/dataflow/data/nontagged/printers/PrintToolsTest.java
11 files changed, 441 insertions(+), 2 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/26/21026/3
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21026?usp=email
To unsubscribe, or for help writing mail filters, visit
https://asterix-gerrit.ics.uci.edu/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: asterixdb
Gerrit-Branch: lumina
Gerrit-Change-Id: I434142a9b9cd2d1fc941b1e1f350e97403a8a3e2
Gerrit-Change-Number: 21026
Gerrit-PatchSet: 3
Gerrit-Owner: Ian Maxon <[email protected]>
Gerrit-Reviewer: Ian Maxon <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Murtadha Hubail <[email protected]>
Gerrit-CC: Anon. E. Moose #1000171
Gerrit-Attention: Ian Maxon <[email protected]>
