arcad/edge
15
0
Fork 0
Code Issues8 Pull Requests Projects Releases3 Wiki Activity

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

Merged
wpetit merged 1 commits from bus-rewrite into master 2023-11-30 15:10:54 +01:00
Conversation 0 Commits 1 Files Changed 50 +1621 -1336
wpetit commented 2023-11-30 15:08:31 +01:00
Owner

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.

Lié à
Corrige

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
wpetit merged commit 870db072e0 into master 2023-11-30 15:10:53 +01:00
wpetit deleted branch bus-rewrite 2023-11-30 15:10:56 +01:00
Sign in to join this conversation.
No Reviewers
No Label
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: arcad/edge#21
No description provided.
Delete Branch "bus-rewrite"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?