async server: need to ignore unary response msg if status is not OK
- Lenguaje dominante
- Java
- Estrellas
- 12.1k
- Forks
- 4k
- Merge medio
- 2 d 17 h
- PR fusionados (30 d)
- 37
Descripción
The expectation for unary responses is that the server will respond with an message or an error. If the server responds with a message and an error, the message should be thrown away. The async API currently would send the message. For unary responses, we should delay sending the response until the onComplete()/onError(). If onError() is called we should not send the message.
https://github.com/grpc/grpc-java/blob/b22017851560197a41015acd90f443f7b9519984/stub/src/main/java/io/grpc/stub/ServerCalls.java#L336-L349
See #5968 and grpc/grpc#12824
Guía de contribución
Línea de trabajo
Comienza en stub/src/main/java/io/grpc/stub/ServerCalls.java, en las líneas 336-349, y sigue el procesamiento de respuestas unary asíncronas. Comprueba el comportamiento alrededor de onComplete() y onError() y verifica después que no se envíe un mensaje cuando lo acompaña un error; usa #5968 y grpc/grpc#12824 como contexto.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- java
- Área
- api, backend
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 45/100