[Bug] QoS count command ignores the times argument after cd
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
Lee CountTelnet#execute y el uso de sus comandos en la implementación QoS de CountTelnet; el issue identifica el análisis de argumentos que selecciona times. Compara los argumentos después de cd con los que se reciben cuando se proporciona el nombre del servicio y, luego de cd repro.Echo, ejecuta o agrega cobertura para count length 3. La tarea estará lista cuando imprima la tabla de estadísticas tres veces, igual que el comando sin cd.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Apache Dubbo Component
Java SDK (apache/dubbo)
Dubbo Version
3.3 branch (cb05763298), OpenJDK 21, macOS. The argument parsing goes back to CountTelnetHandler in 2.0.7 and was carried over to the QoS CountTelnet in #8175.
Steps to reproduce this issue
- Start a provider with QoS enabled and a dubbo service (here
repro.Echowith a methodlength). - Connect to the QoS port and run
cd repro.Echo, thencount length 3.
The statistics table is printed once. Without cd, count repro.Echo length 3 prints it three times, one second apart.
CountTelnet#execute reads the times like this:
if (service == null || service.length() == 0) {
service = args[0];
method = args.length > 1 ? args[1] : null;
} else {
method = args.length > 0 ? args[0] : null;
}
if (StringUtils.isNumber(method)) {
times = method;
method = null;
} else {
times = args.length > 2 ? args[2] : "1";
}
After cd, the arguments are [method, times], so args[2] does not exist and times falls back to "1". (count length 3 5 after cd runs five times.)
What you expected to happen
count length 3 after cd repro.Echo prints the table three times, like count repro.Echo length 3. The usage message of the command also describes cd XxxService as an alternative to giving the service name (count XxxService xxxMethod 10 ... or "cd XxxService" firstly).
Anything else
count 3 after cd (no method) already works, because a numeric first argument is taken as the times.
Are you willing to submit a pull request to fix on your own?
- Yes I am willing to submit a pull request on my own!
- Lenguaje dominante
- Java
- Estrellas
- 41.6k
- Forks
- 26.3k
- Merge medio
- 8 h 6 min
- PR fusionados (30 d)
- 8
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/dubbo
-
[Bug] DefaultSerializationExceptionWrapper does not implement Cleanable, so cleanup() is never invoked by codecsPosiblemente ocupada @binbin0325 la tomó hace 1 día. Abiertocomponent/need-triage type/need-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
apache/dubbo#16537 · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
[Bug] AccessLogFilter accesslog path warnings are garbled and name the wrong accesslog.fixed.path valuePosiblemente ocupada @CCCLL la tomó hace 5 días. Abierto
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
[Bug] dubbo.protocol.triple.max-message-size is only read from OS environment variablesPosiblemente ocupada @CCCLL la tomó hace 5 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
[Bug] StreamUtils treats InputStream.available() as the remaining stream lengthPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertocomponent/need-triage type/need-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
apache/dubbo#16510 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
[Bug] Negative frame data length causes IllegalArgumentException in ExchangeCodecPosiblemente ocupada @L1nq0 la tomó hace 50 días. Abiertocomponent/need-triage type/need-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
Todos los issues de apache/dubbo
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
-
[Bug]: REST API URL starting with "http" but no scheme (e.g. httpbin.org/get) is not given an http:// prefixPosiblemente ocupada @ItsDeadlyProgrammer la tomó hace 1 día. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
appsmithorg/appsmith#42471 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
code-maintenance
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
-
bug priority: medium Review effort [1-5]: 3 status: needs-triage type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
carlos-emr/carlos#5258 ·
Los mantenedores suelen responder en 1 día