[tarantool-patches] Re: [PATCH v1 1/1] sql: refactor vdbe_emit_open_cursor calls
- From: Vladislav Shpilevoy <v.shpilevoy@xxxxxxxxxxxxx>
- To: Kirill Shcherbatov <kshcherbatov@xxxxxxxxxxxxx>, tarantool-patches@xxxxxxxxxxxxx
- Date: Tue, 10 Jul 2018 18:56:24 +0300
2. Why vdbe_emit_open_cursor opens the cursor using index_id,
but opcode OP_OpenRead/Write still uses it as tnum?
It is much more convenient to do this as a part of next commit.
I temporary put this changes as a commit at head
"sql: uniform OP_Open{Write,Read} ops to use tnum"
Maybe it is more convenient, but it is not more correct. Your
first commit still does not fix the mess with index_id/tnum.
Please, squash it into your first commit.
Other related posts:
