diff --git a/src/box/alter.cc b/src/box/alter.cc index 64af71c615555efc6e2c416c635e6ba714d9425e..d73679fb82a87fd289b8bdc1912c8aa11728fbf9 100644 --- a/src/box/alter.cc +++ b/src/box/alter.cc @@ -4712,10 +4712,17 @@ on_replace_dd_space_sequence(struct trigger * /* trigger */, void *event) diag_set(ClientError, ER_NO_SUCH_SEQUENCE, int2str(sequence_id)); return -1; } - + if (stmt->new_tuple != NULL && stmt->old_tuple != NULL) { + /* + * Makes no sense to support update, it would + * complicate the code, and won't simplify + * anything else. + */ + diag_set(ClientError, ER_UNSUPPORTED, + "space \"_space_sequence\"", "update"); + return -1; + } enum priv_type priv_type = stmt->new_tuple ? PRIV_C : PRIV_D; - if (stmt->new_tuple && stmt->old_tuple) - priv_type = PRIV_A; /* Check we have the correct access type on the sequence. * */ if (is_generated || !stmt->new_tuple) { diff --git a/test/box/sequence.result b/test/box/sequence.result index 917952909f2810e6d71535aa5025b49ee78ca029..0a6cfee2c49be8c572bc8931e54a5892432ed067 100644 --- a/test/box/sequence.result +++ b/test/box/sequence.result @@ -2270,3 +2270,22 @@ sq:next() -- 2 sq:drop() --- ... +-- +-- Update on _space_sequence is forbidden. +-- +s = box.schema.create_space('test') +--- +... +pk = s:create_index('pk', {sequence = true}) +--- +... +t = box.space._space_sequence:get({s.id}) +--- +... +box.space._space_sequence:update({s.id}, {{'=', 2, t[2]}}) +--- +- error: space "_space_sequence" does not support update +... +s:drop() +--- +... diff --git a/test/box/sequence.test.lua b/test/box/sequence.test.lua index b0ec020ed7024f1f97c97f1a781603d9ffbdb14c..8e00571e50d00e57dc268ee6196ebcdb52addf41 100644 --- a/test/box/sequence.test.lua +++ b/test/box/sequence.test.lua @@ -771,3 +771,12 @@ sq:next() -- 1 box.begin() box.space._sequence_data:delete{sq.id} box.rollback() sq:next() -- 2 sq:drop() + +-- +-- Update on _space_sequence is forbidden. +-- +s = box.schema.create_space('test') +pk = s:create_index('pk', {sequence = true}) +t = box.space._space_sequence:get({s.id}) +box.space._space_sequence:update({s.id}, {{'=', 2, t[2]}}) +s:drop()