[Vector][ExtensionType] ComplexCopier requires that extension types' holder readers implement getField

Đang mở
#1,109 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

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

Hướng nghiên cứu

Bắt đầu bằng cách đọc ComplexCopier.copy và so sánh các triển khai getField trong UuidReaderImpl và VariantReaderImpl với NullableUuidHolderReaderImpl và NullableVariantHolderReaderImpl. Tái hiện NPE bằng một extension reader dựa trên holder, sau đó xác minh rằng việc sao chép không còn yêu cầu một triển khai getField không phù hợp cho các holder.

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

Mô tả

Type: bug
Describe the bug, including details regarding any error messages, version, and platform.

https://github.com/apache/arrow-java/pull/892 made efforts to streamline and simplify extension readers and writers. In this, ComplexCopier.copy relies on FieldReader.getField being defined for extension types.

Usages of ComplexCopier.copy with readers backed by vector, e.g., UuidReaderImpl and VariantReaderImpl that do have getField implementation, should be OK (as there is concrete notion of Field). However, if holder readers, namely, NullableUuidHolderReaderImpl or NullableVariantHolderReaderImpl are used this results in an NPE.

Having said that getField method for holder readers appears to be a misnomer as we do not have a notion of a field for the holders. This can be seen from the absence of getField overrides for primitive types.

Ngôn ngữ chính
Java
Star
95
Fork
154
Merge trung bình
2 ngày 10 giờ
Pull request đã merge (30 ngày)
11

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

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

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của apache/arrow-java

Tất cả issue của apache/arrow-java

Issue tương tự

Thêm issue về Java

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.