Array[UInt16] forcibly converted to short[]

[guosk123]

Describe the bug

Steps to reproduce

1. create table

clickhouse-server：24.3.12.75

CREATE TABLE IF NOT EXISTS test.t_flow_test
(
 `timestamp` DateTime64(3, 'UTC'),
`src_ip` Array(UInt16),
`dst_ip` UInt16
)
ENGINE = MergeTree
PARTITION BY toYYYYMMDD(timestamp)
ORDER BY (timestamp);

2. use jdbc vcersion

<dependency>
    <groupId>com.clickhouse</groupId>
    <artifactId>clickhouse-jdbc</artifactId>
    <version>0.6.1</version>
</dependency>

3. search in my code

insert into t_flow_test values (NOW(), [50001,50002], 51443);

select src_ip, dst_ip from t_flow_test limit 1;

result：
src_ip, dst_ip
[-15535, -15534], 51443

When I debug it, i found that UInt16 is converted to UnsignedShort in the code, but Array[UInt16] is converted to short[] in the code.

[chernser]

Good day, @guosk123!
Thank you for reporting!

[chernser]

Good day, @guosk123!
This is inconsistency issue. We might not fix it in the old JDBC implementation because a new one is around the corner.
New JDBC implementation will use short because we moved away from custom numeric data type as they are not portable. In most cases JDK provided types will be used.

[guosk123]

Good day, @guosk123! This is inconsistency issue. We might not fix it in the old JDBC implementation because a new one is around the corner. New JDBC implementation will use short because we moved away from custom numeric data type as they are not portable. In most cases JDK provided types will be used.

Okay, I got it

[mshustov]

New JDBC implementation will use short because we moved away from custom numeric data type as they are not portable. In most cases JDK provided types will be used.

@chernser if it's the expected behaviour, should we close as won't fix?