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

Use entrySet() instead of keySet()+get() in WARCSpout and Solr IndexerBolt

Đã đóng Phù hợp với người mới
#2,264 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

@GitMehdi-sys đang làm issue này rồi.

Từ ngày 25/10/2026.

  • #2265 của @GitMehdi-sys — đã merge

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
82/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
java
Lĩnh vực
backend

Hướng nghiên cứu

Kiểm tra external/warc/.../WARCSpout.java trong open() và external/solr/.../bolt/IndexerBolt.java trong execute(). Tìm các vòng lặp duyệt qua keySet() và gọi get() cho từng khóa, sau đó kiểm tra cách mỗi vòng lặp sử dụng các giá trị trong map. Hoàn tất khi cả hai vòng lặp dùng entrySet() và hành vi hiện tại không thay đổi.

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

Mô tả

SpotBugs reports WMI_WRONG_MAP_ITERATOR in two places, where the code
iterates keySet() and then calls get() for each key:

  • external/warc/.../WARCSpout.java (open(), reading the HDFS config map)
  • external/solr/.../bolt/IndexerBolt.java (execute(), iterating metadata)

Iterating entrySet() avoids the extra lookup per entry. The change has no
behavior impact. Would you accept a small PR for this? Happy to do it.

Ngôn ngữ chính
Java
Star
1k
Fork
295
Merge trung bình
1 ngày 14 giờ
Pull request đã merge (30 ngày)
62

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/stormcrawler

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

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.