Skip to content

[client-v2,jdbc-v2] Reading Array(Nullable(String)) with BinaryString enabled #2953

Description

@chernser

Description

There is a claim:

Nullable array strings wrong
High Severity

It looks like readArray is using the top-level readValue for nullable array elements when binaryStringSupport is enabled. This means nested String/FixedString inside Array(Nullable(...)) are read as StringValue instead of plain String. This goes against our rule for container types and could lead to ArrayStoreException or unexpected type mismatches.

https://github.com/ClickHouse/clickhouse-java/pull/2952/changes/BASE..c74bd36aae7fd863f0691654f65c87cc9c982415#r3632548114

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions