diff --git a/src/box/alter.cc b/src/box/alter.cc index 0f8ba082a32f6c0ffa033818af1af4cbab68f9ed..720d2cdd67666fd5537f510272d197387a50cb63 100644 --- a/src/box/alter.cc +++ b/src/box/alter.cc @@ -3834,9 +3834,17 @@ on_replace_dd_space_sequence(struct trigger * /* trigger */, void *event) struct space *space = space_cache_find_xc(space_id); struct sequence *seq = sequence_cache_find(sequence_id); + 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. + */ + tnt_raise(ClientError, ER_UNSUPPORTED, + "space \"_space_sequence\"", "update"); + } + 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()