Réécriture du package bus pour éviter les deadlocks #21

wpetit · 2023-11-30T15:08:31+01:00

wpetit commented

2023-11-30 15:08:31 +01:00

Cette PR introduit une réécriture en profondeur du package bus qui est au coeur des échanges entre les applications et leurs backends.

L'ancienne implémentation souffrait de problèmes de "deadlock" dans certains cas spécifiques (lorsque le context.Context passé à la méthode bus.Request() avait "expiré" ¹). Cette implémentation en est exempt, comme le prouve les tests intégrés.

Plusieurs améliorations annexes liées à la réécriture sont également intégrées dans les changements.

Lié à #20
Corrige #10

How to use Contexts in Go ↩︎

Cette PR introduit une réécriture en profondeur du package `bus` qui est au coeur des échanges entre les applications et leurs backends. L'ancienne implémentation souffrait de problèmes de "deadlock" dans certains cas spécifiques (lorsque le `context.Context` passé à la méthode `bus.Request()` avait "expiré" [^1]). Cette implémentation en est exempt, comme le prouve les tests intégrés. Plusieurs améliorations annexes liées à la réécriture sont également intégrées dans les changements. [^1]: [How to use Contexts in Go](https://www.digitalocean.com/community/tutorials/how-to-use-contexts-in-go) Lié à https://forge.cadoles.com/arcad/edge/issues/20 Corrige https://forge.cadoles.com/arcad/edge/issues/10

wpetit added 1 commit 2023-11-30 15:08:33 +01:00

feat: rewrite bus to prevent deadlocks

arcad/edge/pipeline/head This commit looks good

Details

arcad/edge/pipeline/pr-master This commit looks good

Details

ad49c1718c

wpetit referenced this issue from a commit

2023-11-30 15:10:53 +01:00

Merge pull request 'Réécriture du package `bus` pour éviter les deadlocks' (#21) from bus-rewrite into master

wpetit merged commit 870db072e0 into master

2023-11-30 15:10:53 +01:00