tomcucinotta / distwalk

Distributed processing emulation tool
GNU General Public License v3.0
2 stars 4 forks source link

dw_node assert #52

Open tomcucinotta opened 2 months ago

tomcucinotta commented 2 months ago

probably, all asserts in dw_node should be turned into some graceful shutdown of the connection, or lead to ignoring or returning an error for individual requests

$ ./dw_node -a parent --nt=2
epoll_ctl() failed while deleting socket: Bad file descriptor
dw_node: connection.c:127: conn_find_sock: Assertion `sock != -1' failed.
Aborted (core dumped)
$ ./dw_client -r 1000 -C 1 --nt 10 --ns 10
[...]
RECV Unexpected error: Connection reset by peer
Error: cannot read received message
Fast-forwarding i by 1 pkts
RECV Unexpected error: Connection reset by peer
Error: cannot read received message
Fast-forwarding i by 1 pkts
Connection to 127.0.0.1:7891 failed: Bad file descriptor
Connection to 127.0.0.1:7891 failed: Bad file descriptor
Connection to 127.0.0.1:7891 failed: Bad file descriptor
[...]
tomcucinotta commented 2 months ago

another one:

./dw_node -a parent --nt=6
dw_node: connection.c:263: conn_next_message: Assertion `m->req_size >= sizeof(message_t) && m->req_size <= BUF_SIZE' failed.
Aborted (core dumped)
./dw_client -r 10000 -C 1 --nt 100
tomcucinotta commented 2 months ago

not seeing this assert anymore in current master, is commit 47723053 perhaps fixing the issue someway? (how?)