ClickHouse / ClickHouse/clickhouse-java

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

Đang mở
#2,953 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
area:data-type client-api-v2 jdbc-v2
Ngôn ngữ chính
Java
Star
1.6k
Fork
636
Merge trung bình
2 ngày 23 giờ
Pull request đã merge (30 ngày)
29

Mô tả

### 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

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

Bắt đầu bằng cách lần theo readArray và việc sử dụng readValue khi binaryStringSupport được bật, tập trung vào Array(Nullable(String)) và các giá trị String hoặc FixedString lồng nhau. Xác nhận rằng container trả về các phần tử String thuần túy thay vì StringValue, đồng thời kiểm tra rằng mảng kết quả tránh được lỗi không tương thích kiểu hoặc ArrayStoreException đã báo cáo.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
java
Lĩnh vực
database
Loại issue
Lỗi
Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức độ hoạt động
Ít trao đổi
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
58/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.