Remove RetriableStream flush workaround
- Langage dominant
- Java
- Étoiles
- 12.1k
- Forks
- 4k
- Merge moyen
- 2 j 17 h
- PR mergées (30 j)
- 37
Description
In https://github.com/grpc/grpc-java/pull/9360 I added this workaround:
https://github.com/grpc/grpc-java/blob/7c73baa450c87e6a6f767d7451c46ca256e5158a/core/src/main/java/io/grpc/internal/RetriableStream.java#L565-L568
A flush() must always follow a write (but other flushes and calls could be in-between). I tried to flush everywhere but it didn't work at the time. See also #9340. It is possible there were interactions with https://github.com/grpc/grpc-java/issues/9547 which prevented it from working and just trying again may work.
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Commencez dans core/src/main/java/io/grpc/internal/RetriableStream.java, au niveau du contournement autour des lignes 565-568. Lisez le contexte de #9340 et #9547, puis déterminez si le contournement peut maintenant être supprimé tout en préservant l’exigence selon laquelle chaque write doit être suivi d’un flush. Le travail est terminé lorsque le contournement est supprimé et que le comportement de flush qui en résulte reste correct.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- java
- Domaine
- api
- Type d'issue
- Refactorisation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100