

<HTML><BODY><div>LGTM<br><br> <blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;">Пятница, 29 мая 2020, 15:41 +03:00 от Alexander Turenko <alexander.turenko@tarantool.org>:<br> <div id=""><div class="js-helper js-readmsg-msg"><style type="text/css"></style><div><div id="style_15907561110863963172_BODY">On Tue, May 19, 2020 at 11:05:25PM +0300, Leonid Vasiliev wrote:<div class="mail-quote-collapse">> The tcp server starts in a separate fiber.<br>> When server's socket is closed from another fiber,<br>> an exception will be thrown in server's loop from<br>> check_socket function.<br>> A "socket is close" check has been added at server's<br>> fiber and now server's fiber terminates correctly.<br>><br>> Fixes: #4087<br>> ---<br>> <a href="https://github.com/tarantool/tarantool/issues/4087" target="_blank">https://github.com/tarantool/tarantool/issues/4087</a><br>> <a href="https://github.com/tarantool/tarantool/tree/lvasiliev/gh-4087-fix-socket-stuff" target="_blank">https://github.com/tarantool/tarantool/tree/lvasiliev/gh-4087-fix-socket-stuff</a><br>><br>> @Changelog fix error while closing socket.tcp_server socket(gh-4087)<br>><br>> For a "socket is close" check, fd < 0 is used instead of an additional flag.</div><br>LGTM.</div></div></div></div></blockquote> <div> </div><div data-signature-widget="container"><div data-signature-widget="content"><div>--<br>Ilya Kosarev</div></div></div><div> </div></div></BODY></HTML>