crud icon indicating copy to clipboard operation
crud copied to clipboard

Update operation convert is broken for splice

Open DifferentialOrange opened this issue 2 years ago • 2 comments
trafficstars

tarantool> box.info.version
---
- 1.10.15-0-ge14d0b4
...

tarantool> crud.update('customers', 1, {{":", "name", 2, 3, "a"}})
---
- null
- line: 177
  class_name: UpdateError
  err: 'Failed to update: Unknown UPDATE operation'
  file: ./crud/update.lua
  str: 'UpdateError: Failed to update: Unknown UPDATE operation'
...

yet it is actually supported by Tarantool 1.10.15. The reason is that the operation is converted to invalid [[":",3,2]] operation: https://github.com/tarantool/crud/blob/48f801c0257f8c948fc8ffaf8992300f814f5b97/crud/common/utils.lua#L776-L778

DifferentialOrange avatar Nov 17 '23 07:11 DifferentialOrange

This one is likely to be closed together with 1.10 support drop

DifferentialOrange avatar Nov 17 '23 08:11 DifferentialOrange

We need to remove a skip helper from the go-tarantool tests after the fix: https://github.com/tarantool/go-tarantool/pull/353

oleg-jukovec avatar Nov 17 '23 10:11 oleg-jukovec