[tarantool-patches] Re: [PATCH] sql: remove preupdate hook

  • From: "n.pettik" <korablev@xxxxxxxxxxxxx>
  • To: tarantool-patches@xxxxxxxxxxxxx
  • Date: Fri, 27 Jul 2018 16:33:50 +0300


I have grepped and found few mentions about preupdate hook
which you didn’t delete.

Firstly, it seems that you can remove struct PreUpdate.
Secondly, one comment in update.c (line 565):

* That (regNew==regnewPk+1) is true is also important for the
* pre-update hook. If the caller invokes preupdate_new(), the returned
* value is copied from memory cell (regNewPk+1+iCol), where iCol
* is the column index supplied by the user.

Remove it as well pls.
Fixed. Updated patch below.

Thx, now LGTM.

Other related posts:

  1. Home
  2. tarantool-patches
  3. Archive
  4. 07-2018