danepitkin commented on code in PR #39188:
URL: https://github.com/apache/arrow/pull/39188#discussion_r1425561173
##########
java/adapter/jdbc/src/test/resources/h2/test1_all_datatypes_h2.yml:
##########
@@ -13,59 +13,59 @@ name: 'test1_all_datatypes_h2'
create: 'CREATE TABLE table1 (int_field1 INT, bool_field2 BOOLEAN,
tinyint_field3 TINYINT, smallint_field4 SMALLINT, bigint_field5 BIGINT,
decimal_field6 DECIMAL(20,2), double_field7 DOUBLE, real_field8 REAL,
time_field9 TIME, date_field10 DATE, timestamp_field11 TIMESTAMP,
- binary_field12 BINARY(100), varchar_field13 VARCHAR(256), blob_field14
BLOB, clob_field15 CLOB, char_field16 CHAR(16), bit_field17 BIT,
- null_field18 NULL, list_field19 ARRAY, map_field20 VARCHAR(256));'
+ binary_field12 VARBINARY(100), varchar_field13 VARCHAR(256), blob_field14
BLOB, clob_field15 CLOB, char_field16 CHAR(14), bit_field17 BIT,
Review Comment:
Yes, `CHAR` is now a fixed length buffer that pads unused memory with
whitespace. Before, it was treated more like a `VARCHAR` where unused space was
not padded and returned. So with `CHAR(16)`, the H2 DB v1.4.196 it returned
`some char text`, but in v2.2.224 it returned `some char text `.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
