[Tarantool-patches] [PATCH] sequence: return correct error on using dropped sequence
Chris Sosnin
k.sosnin at tarantool.org
Wed Feb 26 19:13:08 MSK 2020
This code is called from C, so it shouldn't throw.
Closes #4753
---
branch: https://github.com/tarantool/tarantool/tree/ksosnin/gh-4753-sequence-cpp-exception
issue: https://github.com/tarantool/tarantool/issues/4753
src/box/schema.cc | 2 +-
test/box/sequence.result | 17 +++++++++++++++++
test/box/sequence.test.lua | 8 ++++++++
3 files changed, 26 insertions(+), 1 deletion(-)
diff --git a/src/box/schema.cc b/src/box/schema.cc
index fab7544f2..456eef429 100644
--- a/src/box/schema.cc
+++ b/src/box/schema.cc
@@ -646,7 +646,7 @@ sequence_cache_find(uint32_t id)
{
struct sequence *seq = sequence_by_id(id);
if (seq == NULL)
- tnt_raise(ClientError, ER_NO_SUCH_SEQUENCE, int2str(id));
+ diag_set(ClientError, ER_NO_SUCH_SEQUENCE, int2str(id));
return seq;
}
diff --git a/test/box/sequence.result b/test/box/sequence.result
index 0a6cfee2c..32a094d6f 100644
--- a/test/box/sequence.result
+++ b/test/box/sequence.result
@@ -2102,6 +2102,23 @@ s:drop()
---
...
--
+-- gh-4753: accessing dropped sequence should yield correct error
+--
+s = box.schema.sequence.create('s')
+---
+...
+s:drop()
+---
+...
+s:next()
+---
+- error: Sequence '1' does not exist
+...
+s:reset()
+---
+- error: Sequence '1' does not exist
+...
+--
-- Check that altering parts of a primary index with a sequence
-- attached requires sequence update. Renaming fields does not.
--
diff --git a/test/box/sequence.test.lua b/test/box/sequence.test.lua
index 8e00571e5..d8a212fab 100644
--- a/test/box/sequence.test.lua
+++ b/test/box/sequence.test.lua
@@ -710,6 +710,14 @@ s.index.pk:alter{sequence = {field = 'x.a.b[1]'}}
s:replace{{a = {b = {box.NULL}}}} -- ok
s:drop()
+--
+-- gh-4753: accessing dropped sequence should yield correct error
+--
+s = box.schema.sequence.create('s')
+s:drop()
+s:next()
+s:reset()
+
--
-- Check that altering parts of a primary index with a sequence
-- attached requires sequence update. Renaming fields does not.
--
2.21.1 (Apple Git-122.3)
More information about the Tarantool-patches
mailing list