[Tarantool-patches] [PATCH v3 3/5] sql: introduce SET statement

[Vladislav Shpilevoy]

On 27/11/2019 11:33, Mergen Imeev wrote:
> Hi! Thanks for the suggestion! I think your points are correct,
> but I think we should create a new issue after pushing SET.
> Issue #4621 about using VDBE in SET/control pragmas. All other
> pragmas already use VDBE. Since SET will use VDBE, I think we
> should close #4621 with #4511.

I see that we have different understanding of what it means - use
VDBE. I mean, that option value should be get/set at runtime.
Currently the only thing runtime does is return of metadata. All
the actual work happens at compile time.

To make binding works the only solution is to make
PRAGMA/SET/whatever work at runtime totally. When you will
properly move PRAGMA/SET to VDBE, you will get bindings for
free.

The patchset LGTM. Perhaps. I don't really know what is
our plan on session and SQL settings, so I can only approve the
technical part.

Seems like in our team design decisions about everything work
like 'lets see what PostgreSQL/MySQL/Ya.DB do, and we will do
literally the same'.