[Tarantool-patches] [PATCH 0/4] session settings fixes
Timur Safin
tsafin at tarantool.org
Wed Apr 8 12:40:38 MSK 2020
: -----Original Message-----
: From: Tarantool-patches <tarantool-patches-bounces at dev.tarantool.org> On
: Behalf Of Chris Sosnin
:
: > On 3 Apr 2020, at 20:09, Peter Gulutzan <pgulutzan at ocelot.ca> wrote:
: >
: > Hi,
: >
: > Re:
: > SETTING SET "sql_default_engine" = 'memtx';
: >
: > Adding a new reserved word SETTING, without advance notice,
: > should be against policy. It causes a slight risk
: > of breaking an existing application.
: >
: > I had hoped for
: > UPDATE _SETTINGS_DEFAULT_ENGINE SET VALUE = 'memtx';
:
: Implementing this syntax would either require creating a table for each
: setting
: or adding new keywords, since this is the same as updating tables.
:
: Moreover, SET VALUE construction also is not supported, so we still need
: to reserve a new word.
:
: We can borrow another MySQL syntax — SET SESSION name = value
: What do you think about this option?
:
Peter, after multiple rounds of discussions here (one incompatible way against another incompatible way) we tend to prefer the simpler syntax
SET SESSION sqlconfigname = value
because it's quite simpler, looks similar to MySQL way (though still is not part of standard), and would require less intrusive patch to parser. Introducing all configuration settins as separate system spaces (via hardcoding in the engine) looked to us slightly more intrusive (while still being not as much closer to the standard way).
Minuses against minuses on both sides.
What do you think, Peter?
Thanks,
Timur
More information about the Tarantool-patches
mailing list