Use entrySet() instead of keySet()+get() in WARCSpout and Solr IndexerBolt
メンテナーはふだん 1 日以内に返信
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 82/100
調査の方向性
external/warc/.../WARCSpout.java の open() と external/solr/.../bolt/IndexerBolt.java の execute() を調べてください。keySet() を反復し、各キーに対して get() を呼び出すループを見つけ、それぞれのループがマップの値をどのように使っているかを確認してください。両方のループが entrySet() を使い、既存の動作が変わらなければ完了です。
索引モデルが issue の本文から書いたものです。
説明
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.
- 主要言語
- Java
- スター
- 1k
- フォーク
- 295
- 平均マージ
- 1日 14時間
- マージ済み PR(30日)
- 62
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
apache/stormcrawler のほかの issue
-
Remove dependency on httpcomponents.core5対応中かも @rzo1 が今日担当しました。 オープンBREAKING CHANGE core
難易度 5/5 1週間以上 初心者へのやさしさ 15/100
apache/stormcrawler#2266 ·
メンテナーはふだん 1 日以内に返信
-
URLNormalizer The query is split on `;` as well as `&`, which rewrites URLs対応中かも @jnioche が 26 日前に担当しました。 オープンbug
apache/stormcrawler#2199 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
[Bug report] BasicURLNormalizer unmangleQueryString inserts a second `?` when a query value contains a `/`対応中かも @jnioche が 26 日前に担当しました。 オープンbug core
apache/stormcrawler#2198 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 70/100
apache/stormcrawler#2131 ·
メンテナーはふだん 1 日以内に返信
-
[Improvement] Work towards Maven 4 Compatibility対応中かも @mvolikas が 351 日前に担当しました。 オープンgood first issue help wanted
apache/stormcrawler#1662 · コメント 2 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
apache/stormcrawler の issue をすべて見る
似ている issue
-
agent:working follow-up
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
Sanne/incus-spawn#1683 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
checkstyle/checkstyle#22210 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 75/100
521xueweihan/HelloGitHub#4105 ·
-
devops good first issue security
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
asciidoctor/asciidoctorj#1339 ·