[Tarantool-patches] [PATCH v2 01/10] sql: do not return UNSIGNED in built-in functions

Vladislav Shpilevoy v.shpilevoy at tarantool.org
Sat Aug 22 17:23:35 MSK 2020


Hi! Thanks for the patch!

On 14.08.2020 17:04, imeevma at tarantool.org wrote:
> This patch forces functions to return INTEGER instead of UNSIGNED.

Why? I don't see a reason why length() would need to return int instead
of uint.

> diff --git a/test/sql/types.result b/test/sql/types.result
> index 442245186..95f7713e8 100644
> --- a/test/sql/types.result
> +++ b/test/sql/types.result
> @@ -2795,3 +2795,15 @@ box.execute([[DROP TABLE ts;]])
>  ---
>  - row_count: 1
>  ...
> +--
> +-- gh-4159: Make sure that functions returns values of type INTEGER
> +-- instead of values of type UNSIGNED.

What are the other functions besides length()?

> +--
> +box.execute([[SELECT typeof(length('abc'));]])


More information about the Tarantool-patches mailing list