[tarantool-patches] Re: [PATCH 5/6] sql: introduce extended range for INTEGER type

[Konstantin Osipov]

* Nikita Pettik <korablev at tarantool.org> [19/06/07 23:30]:
> @@ -72,11 +72,11 @@ execute('SELECT ?, ?, ?', {1, 2, 3})
>  ---
>  - metadata:
>    - name: '?'
> -    type: INTEGER
> +    type: UNSIGNED
>    - name: '?'
> -    type: INTEGER
> +    type: UNSIGNED
>    - name: '?'
> -    type: INTEGER
> +    type: UNSIGNED
>    rows:
>    - [1, 2, 3]

Nikita,

I know this decision came after you coded most of the patch, but
we agreed that unsigned integer literals should still have
"INTEGER" type, not "UNSIGNED", since INTEGER in tarantool fully
covers UNSIGNED range.

This leads to fewer surprises to users, e.g. when calculating
expression types. Could you please fix this?

> 

-- 
Konstantin Osipov, Moscow, Russia