Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

[C++] GetSchema labels its null check on each schema field as "DictionaryEncoding.indexType"

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#51,780 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@CaptainAni187 đang làm issue này rồi.

Từ ngày 6/10/2026.

  • #51811 của @CaptainAni187 — đang mở

Đánh giá

Độ khó
1/5
Thời gian dự kiến
Dưới một giờ
Mức phù hợp với người mới
88/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
cpp
Lĩnh vực
data

Hướng nghiên cứu

Mở cpp/src/arrow/ipc/metadata_internal.cc quanh vòng lặp GetSchema (các dòng được liên kết ~1450–1457). CHECK_FLATBUFFERS_NOT_NULL trên mỗi field dùng nhãn DictionaryEncoding.indexType; hoặc bỏ kiểm tra đó (như comment gần đó gợi ý) hoặc gắn lại nhãn thành thứ gì đó như Schema.fields[i]. Kiểm tra DictionaryEncoding.indexType thực sự vẫn ở ~901. Xong khi vòng lặp đó không còn báo sai tên field.

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

Mô tả

Component: C++

[!NOTE]
I discovered this issue and wrote it up with help from Claude Opus 5.5.

Describe the enhancement requested

In GetSchema in cpp/src/arrow/ipc/metadata_internal.cc, the loop over a schema's fields checks each field for null with the wrong label:

for (int i = 0; i < num_fields; ++i) {
  const flatbuf::Field* field = schema->fields()->Get(i);
  // XXX I don't think this check is necessary (AP)
  CHECK_FLATBUFFERS_NOT_NULL(field, "DictionaryEncoding.indexType");
  RETURN_NOT_OK(
      FieldFromFlatbuffer(field, field_pos.child(i), dictionary_memo, &fields[i]));
}

The value being checked is a Field, not DictionaryEncoding.indexType. If the check ever failed, it would report Unexpected null field DictionaryEncoding.indexType in flatbuffer-encoded metadata. That message already belongs to the real check on dictionary index types at line 901, so the two errors would be indistinguishable.

As the comment beside it suggests, the check also appears unable to fail. For a vector of tables, the vendored flatbuffers IndirectHelper<Offset<T>>::Read returns the element's address plus its stored offset, so fields()->Get(i) doesn't return null. The function already checks schema->fields() itself for null, with the correct label Schema.fields.

Either option would fix it:

  • Remove the check, as the comment suggests.
  • Keep it with an accurate label, such as "Schema.fields[i]".

I found this while looking into how the reader handles a missing DictionaryEncoding.indexType, which is reported separately at #51779.

Ngôn ngữ chính
C++
Star
17.2k
Fork
4.3k
Merge trung bình
4 ngày 5 giờ
Pull request đã merge (30 ngày)
101

Chuẩn bị môi trường

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

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

Issue tương tự

Thêm issue về C++

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.