[Tarantool-patches] [PATCH 0/2] sql: revoke table privileges on drop

Chris Sosnin k.sosnin at tarantool.org
Wed Dec 18 14:00:57 MSK 2019


Hi! Thank you for the review and for your fixes!

> +box.schema.user.drop('test_user1')
> +---
> +...
> +box.schema.user.drop('test_user2')
> +---
> +...
> +box.space._sequence:select{}
> +---
> +- []
> +...

I moved sequence tests to the second commit, firstly, because here you
don't do seq:drop() (I believe because after my initial patch sequences
are getting deleted anyways, but after yours seq remains in _sequence
space) and, secondly, because this way we can better isolate these two patches
from each other.

I agree with the rest, especially with your catch about sequences.
Thus, I'm sending this patchset to Nikita.

branch: https://github.com/tarantool/tarantool/tree/ksosnin/gh-4546-sql-drop-grants
issue: https://github.com/tarantool/tarantool/issues/4546

Chris Sosnin (2):
  sql: remove grants associated with the table
  sql: drop only generated sequence in DROP TABLE

 src/box/sql/build.c                       | 81 +++++++++++++++++++----
 src/box/user.cc                           |  7 ++
 src/box/user.h                            |  4 ++
 test/sql/autoincrement.result             | 80 ++++++++++++++++++++++
 test/sql/autoincrement.test.lua           | 33 +++++++++
 test/sql/gh-4546-sql-drop-grants.result   | 54 +++++++++++++++
 test/sql/gh-4546-sql-drop-grants.test.lua | 24 +++++++
 7 files changed, 269 insertions(+), 14 deletions(-)
 create mode 100644 test/sql/autoincrement.result
 create mode 100644 test/sql/autoincrement.test.lua
 create mode 100644 test/sql/gh-4546-sql-drop-grants.result
 create mode 100644 test/sql/gh-4546-sql-drop-grants.test.lua

-- 
2.21.0 (Apple Git-122.2)



More information about the Tarantool-patches mailing list