Use entrySet() instead of keySet()+get() in WARCSpout and Solr IndexerBolt
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 82/100
Línea de trabajo
Inspecciona external/warc/.../WARCSpout.java en open() y external/solr/.../bolt/IndexerBolt.java en execute(). Busca los bucles que iteran sobre keySet() y llaman a get() para cada clave; después, comprueba cómo usa cada bucle los valores del mapa. La tarea estará terminada cuando ambos bucles usen entrySet() y el comportamiento existente no haya cambiado.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Java
- Estrellas
- 1k
- Forks
- 295
- Merge medio
- 1 d 14 h
- PR fusionados (30 d)
- 62
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de apache/stormcrawler
-
Remove dependency on httpcomponents.core5Posiblemente ocupada @rzo1 la tomó hace 1 día. AbiertoBREAKING CHANGE core
Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100
apache/stormcrawler#2266 ·
Los mantenedores suelen responder en 1 día
-
URLNormalizer The query is split on `;` as well as `&`, which rewrites URLsPosiblemente ocupada @jnioche la tomó hace 27 días. Abiertobug
apache/stormcrawler#2199 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
[Bug report] BasicURLNormalizer unmangleQueryString inserts a second `?` when a query value contains a `/`Posiblemente ocupada @jnioche la tomó hace 27 días. Abiertobug core
apache/stormcrawler#2198 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 70/100
apache/stormcrawler#2131 ·
Los mantenedores suelen responder en 1 día
-
[Improvement] Work towards Maven 4 CompatibilityPosiblemente ocupada @mvolikas la tomó hace 352 días. Abiertogood first issue help wanted
apache/stormcrawler#1662 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
Todos los issues de apache/stormcrawler
Issues similares
-
[BUG] jackson3 JacksonJsonpMapper(ObjectMapper) fails to deserialize `_source` with "Trailing token" errorPosiblemente ocupada @junyeokk la tomó hoy. Abiertobug untriaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
opensearch-project/opensearch-java#2236 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
open-learning-exchange/myplanet#18392 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
linagora/twake-calendar-side-service#1218 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
[Bug] [Go] Ack returns nil when the server answers with a non-OK statusPosiblemente ocupada @leizhiyuan la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
apache/rocketmq-clients#1465 ·
Los mantenedores suelen responder en 2 días