1
0
Fork
You've already forked gorram
0

Ensure server actually exits #39

Open
opened 2019年11月08日 23:18:20 +01:00 by aqtrans · 1 comment
aqtrans commented 2019年11月08日 23:18:20 +01:00 (Migrated from git.jba42.com)
Copy link

Need to ensure the cancel context call/signal is passed to all spawned goroutines.

Specifically all deadClientTickers and such.
Easy to trigger: after benchmarking, these will continue running even after ctrl+c is caught.

Need to ensure the cancel context call/signal is passed to all spawned goroutines. Specifically all deadClientTickers and such. Easy to trigger: after benchmarking, these will continue running even after ctrl+c is caught.
aqtrans commented 2019年11月08日 23:18:50 +01:00 (Migrated from git.jba42.com)
Copy link
main.(*gorramServer).Ping(0xc0000a4fc0, 0xa78120, 0xc0004a7950, 0xc00055b6e0, 0xc0000a4fc0, 0xc0001d3898, 0x46b0cc)
 /home/aqtrans/Personal/go/src/git.jba.io/go/gorram/server/main.go:187 +0x392
``` main.(*gorramServer).Ping(0xc0000a4fc0, 0xa78120, 0xc0004a7950, 0xc00055b6e0, 0xc0000a4fc0, 0xc0001d3898, 0x46b0cc) /home/aqtrans/Personal/go/src/git.jba.io/go/gorram/server/main.go:187 +0x392 ```
Sign in to join this conversation.
No Branch/Tag specified
master
cfg-dir
mute-alerts
keepalive
alert-filtering
goroutines
No results found.
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
aqtrans/gorram#39
Reference in a new issue
aqtrans/gorram
No description provided.
Delete branch "%!s()"

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?