feat(wamp-server) #76: optional timeout in callBlocking #77
Conversation
There was a problem hiding this comment.
Il y a un problème sur la gestion de la concurrence, à traiter. Pour le reste c'est bon, il faudra penser par contre à indiquer dans la version de la lib qu'il y a un breaking change dans l'api, les valeurs par défaut sont gérées uniquement à la compilation et en kotlin, si on a un code java ou si on ne recompile pas un code kotlin qui utilise le callBlocking ou sendBlocking ça ne marchera pas. Ca me parait acceptable il faut juste l'indiquer dans les release notes.
| while (currentCall != null && (Clock.System.now() - now).inWholeMilliseconds < timeoutInMs) { | ||
| val capturedCurentCall = currentCall | ||
| while ( | ||
| capturedCurentCall != null && |
There was a problem hiding this comment.
on change le comportement ici, on ne vérifie plus le currentCall à chaque iteration, mais uniquement le capturedCurrentCall avant l'entrée dans la boucle. Et donc on ne sortira pas de la boucle si un event concurrent met le call à null.
If unset, use default one given by wamp server settings
lilgallon
force-pushed
the
feat/#76-optional-timeout-send-blocking
branch
from
f8101d9
to
749fc10
Compare
|
If unset, use default one given by wamp server settings
closes #76