

<HTML><BODY><div>LGTM<br> <blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;">Пятница, 10 апреля 2020, 17:18 +03:00 от Sergey Bronnikov <sergeyb@tarantool.org>:<br> <div id=""><div class="js-helper js-readmsg-msg"><style type="text/css"></style><div><div id="style_15865283291135785115_BODY">LGTM<br><br>On 10:11 Wed 08 Apr , Alexander V. Tikhonov wrote:<div class="mail-quote-collapse">> Found that time limit in function used for testing in sql/func-recreate<br>> test not enough for some time. Fiber set next is too near the time<br>> limit. Decided to increase the time limit of the function, anyway it<br>> checks its value at the test.<br>><br>> Close #4384<br>> ---<br>><br>> Github: <a href="https://github.com/tarantool/tarantool/tree/avtikhon/gh-4384-increase-waitfor-time" target="_blank">https://github.com/tarantool/tarantool/tree/avtikhon/gh-4384-increase-waitfor-time</a><br>> Issue: <a href="https://github.com/tarantool/tarantool/issues/4384" target="_blank">https://github.com/tarantool/tarantool/issues/4384</a><br>><br>> test/sql/func-recreate.result | 6 +++---<br>> test/sql/func-recreate.test.lua | 2 +-<br>> test/sql/suite.ini | 1 -<br>> 3 files changed, 4 insertions(+), 5 deletions(-)<br>><br>> diff --git a/test/sql/func-recreate.result b/test/sql/func-recreate.result<br>> index 73fb03cc4..c28e2c071 100644<br>> --- a/test/sql/func-recreate.result<br>> +++ b/test/sql/func-recreate.result<br>> @@ -18,7 +18,7 @@ box.internal.sql_create_function('WAITFOR', 'INT', function (n) fiber.sleep(n) r<br>> ch = fiber.channel(1)<br>> ---<br>> ...<br>> -_ = fiber.create(function () ch:put(box.execute('select WAITFOR(0.2)')) end)<br>> +_ = fiber.create(function () ch:put(box.execute('select WAITFOR(0.5)')) end)<br>> ---<br>> ...<br>> fiber.sleep(0.1)<br>> @@ -32,10 +32,10 @@ box.internal.sql_create_function('WAITFOR', 'INT', function (n) require('fiber')<br>> ch:get()<br>> ---<br>> - metadata:<br>> - - name: WAITFOR(0.2)<br>> + - name: WAITFOR(0.5)<br>> type: integer<br>> rows:<br>> - - [0.2]<br>> + - [0.5]<br>> ...<br>> box.internal.sql_create_function('WAITFOR', 'INT', function (n) require('fiber').sleep(n) return n end)<br>> ---<br>> diff --git a/test/sql/func-recreate.test.lua b/test/sql/func-recreate.test.lua<br>> index 753e9ca4d..fa480288f 100644<br>> --- a/test/sql/func-recreate.test.lua<br>> +++ b/test/sql/func-recreate.test.lua<br>> @@ -8,7 +8,7 @@ box.internal.sql_create_function('WAITFOR', 'INT', function (n) fiber.sleep(n) r<br>><br>> ch = fiber.channel(1)<br>><br>> -_ = fiber.create(function () ch:put(box.execute('select WAITFOR(0.2)')) end)<br>> +_ = fiber.create(function () ch:put(box.execute('select WAITFOR(0.5)')) end)<br>> fiber.sleep(0.1)<br>><br>> box.internal.sql_create_function('WAITFOR', 'INT', function (n) require('fiber').sleep(n) return n end)<br>> diff --git a/test/sql/suite.ini b/test/sql/suite.ini<br>> index a8664c510..cc1d641a8 100644<br>> --- a/test/sql/suite.ini<br>> +++ b/test/sql/suite.ini<br>> @@ -11,4 +11,3 @@ release_disabled = errinj.test.lua view_delayed_wal.test.lua sql-debug.test.lua<br>> disabled = sql-statN-index-drop.test.lua<br>> pretest_clean = True<br>> fragile = dll.test.lua ; gh-4427<br>> - func-recreate.test.lua ; gh-4384<br>> --<br>> 2.17.1<br>></div><br>--<br>sergeyb@</div></div></div></div></blockquote> <div> </div><div data-signature-widget="container"><div data-signature-widget="content"><div>--<br>Oleg Piskunov</div></div></div><div> </div></div></BODY></HTML>