sql: rework error handling in box.execute()
In accordance with the Lua coding style in Tarantool, all errors returned in Lua should be returned using 'return nil, error'. However, box.execute() throws an exception in case of an error. This patch causes box.execute() to return an error, as described in the coding style. Closes #4390
Showing
- src/box/lua/execute.c 2 additions, 2 deletionssrc/box/lua/execute.c
- test/sql-tap/gh2548-select-compound-limit.test.lua 5 additions, 5 deletionstest/sql-tap/gh2548-select-compound-limit.test.lua
- test/sql-tap/lua/sqltester.lua 4 additions, 1 deletiontest/sql-tap/lua/sqltester.lua
- test/sql/bind.result 11 additions, 3 deletionstest/sql/bind.result
- test/sql/bind.test.lua 7 additions, 1 deletiontest/sql/bind.test.lua
- test/sql/checks.result 34 additions, 18 deletionstest/sql/checks.result
- test/sql/clear.result 6 additions, 3 deletionstest/sql/clear.result
- test/sql/collation.result 42 additions, 21 deletionstest/sql/collation.result
- test/sql/ddl.result 15 additions, 15 deletionstest/sql/ddl.result
- test/sql/ddl.test.lua 15 additions, 15 deletionstest/sql/ddl.test.lua
- test/sql/delete.result 12 additions, 7 deletionstest/sql/delete.result
- test/sql/drop-table.result 10 additions, 6 deletionstest/sql/drop-table.result
- test/sql/errinj.result 30 additions, 15 deletionstest/sql/errinj.result
- test/sql/foreign-keys.result 16 additions, 11 deletionstest/sql/foreign-keys.result
- test/sql/gh-2929-primary-key.result 10 additions, 5 deletionstest/sql/gh-2929-primary-key.result
- test/sql/gh-2981-check-autoinc.result 8 additions, 4 deletionstest/sql/gh-2981-check-autoinc.result
- test/sql/gh-3613-idx-alter-update.result 2 additions, 1 deletiontest/sql/gh-3613-idx-alter-update.result
- test/sql/gh-3888-values-blob-assert.result 12 additions, 6 deletionstest/sql/gh-3888-values-blob-assert.result
- test/sql/icu-upper-lower.result 6 additions, 3 deletionstest/sql/icu-upper-lower.result
- test/sql/insert-unique.result 4 additions, 2 deletionstest/sql/insert-unique.result
Loading
Please register or sign in to comment