[Tarantool-patches] [PATCH 4/5] sql: replace control pragmas by SET

[Mergen Imeev]

On Wed, Nov 27, 2019 at 02:39:17PM +0300, Konstantin Osipov wrote:
> * Mergen Imeev <imeevma at tarantool.org> [19/11/27 14:27]:
> > > But how are you going to fix it? By making SET statement powerful
> > > enough to query a value or by making the sysview updatable?
> > > 
> > Since all settings are saved in a “struct session”, I
> > think that I will use the same methods PRAGMA used. The
> > system view reads from current_session(), so SET will
> > also read from there.
> 
> OK, so you are going to allow SET to return an existing value,
> without setting it. Then what is the point of having a view,
> if you can both get and set the values using SET statement?
> 
I agree that _vsession_settings can be moved from this
issue to another. It is here due to historical reasons -
at first it was called _vsql_settings and served as the
only way to get SQL setting values. Even now, even if this
is still the only way to get the values, it can be
transferred to another issue. If we allow to get the
setting value using the SET statement, then this is
definitely worth doing.

However, the basic concept of _vsession_settings is
slightly different. As I said earlier, it allows you to
get the current values of the session settings from all
interfaces.

> -- 
> Konstantin Osipov, Moscow, Russia