Skip to content

Commit

Permalink
crud: allow tuple fields of select options to be cdata in type checks
Browse files Browse the repository at this point in the history
In scope of tarantool/tarantool#8147, box tuples returned from remote
procedure calls our now encoded as a new extension, `MP_TUPLE`, and, as,
the results of such calls are now decoded as box tuples, i.e., cdata. For
compatibility with this feature, we need to allow tuple fields of select
options, which are returned from a remote procedure calls, to be cdata in
option type checks.

Needed for tarantool/tarantool#8147
  • Loading branch information
CuriousGeorgiy authored and DifferentialOrange committed Oct 23, 2023
1 parent d799d57 commit 241cdee
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 4 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.

### Fixed
* Compatibility with Tarantool 3.0 binary protocol change (#390).
* Compatibility with Tarantool 3.0 tuple objects (#390).

## [1.4.0] - 16-10-23

Expand Down
4 changes: 2 additions & 2 deletions crud/select.lua
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,8 @@ end

local function select_on_storage(space_name, index_id, conditions, opts)
dev_checks('string', 'number', '?table', {
scan_value = 'table',
after_tuple = '?table',
scan_value = 'table|cdata',
after_tuple = '?table|cdata',
tarantool_iter = 'number',
limit = 'number',
scan_condition_num = '?number',
Expand Down
4 changes: 2 additions & 2 deletions crud/select/executor.lua
Original file line number Diff line number Diff line change
Expand Up @@ -76,8 +76,8 @@ end

function executor.execute(space, index, filter_func, opts)
dev_checks('table', 'table', 'function', {
scan_value = 'table',
after_tuple = '?table',
scan_value = 'table|cdata',
after_tuple = '?table|cdata',
tarantool_iter = 'number',
limit = '?number',
yield_every = '?number',
Expand Down

0 comments on commit 241cdee

Please sign in to comment.