[tarantool-patches] Re: [PATCH 4/6] sql: enforce implicit type conversions

Vladislav Shpilevoy v.shpilevoy at tarantool.org
Tue Oct 30 00:32:52 MSK 2018


Thanks for the fixes! See 3 comments below.

> --- a/test/sql-tap/boundary1.test.lua
> +++ b/test/sql-tap/boundary1.test.lua
> @@ -7649,5 +7649,4 @@ test:do_execsql_test(
>      "boundary1-2.66.le.5",
>      "SELECT a FROM t1 WHERE rowid <= -9.22337303685477580800e+18 ORDER BY x",
>      {})
> -
>  test:finish_test(
1. Stray diff in boundary1.test.lua.

>>> diff --git a/test/sql-tap/boundary2.test.lua b/test/sql-tap/boundary2.test.lua
>>> index 3eaef75dc..be4b8750d 100755
>>> --- a/test/sql-tap/boundary2.test.lua
>>> +++ b/test/sql-tap/boundary2.test.lua
>>> @@ -1,6 +1,6 @@
>>>   #!/usr/bin/env tarantool
>>>   test = require("sqltester")
>>> -test:plan(3021)
>>> +test:plan(2965)
>>>     --!./tcltestrunner.lua
>>>   -- 2008 December 11
>>> @@ -7462,6 +7462,7 @@ test:do_execsql_test(
>>>       "SELECT a FROM t1 WHERE r > 9.22337303685477580800e+18 ORDER BY a DESC",
>>>       {})
>>>   +if false then
>>
>> 6. I thought you have removed all these 'if false'. Please, do it.
> 
> I did it in boundary3 test and forgot about this one. Fixed:

2. Sorry, still can be found in cast.test.lua.

Also stray empty lines on 526, 763.

3. if false in in3.test.lua.





More information about the Tarantool-patches mailing list