Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

box.execute doesn't support bind parameters with type 'decimal' #4717

Closed
knazarov opened this issue Jan 7, 2020 · 1 comment
Closed

box.execute doesn't support bind parameters with type 'decimal' #4717

knazarov opened this issue Jan 7, 2020 · 1 comment
Assignees
Labels
feature A new functionality sql
Milestone

Comments

@knazarov
Copy link

knazarov commented Jan 7, 2020

How to reproduce:

#!/usr/bin/env tarantool

local decimal = require('decimal')

box.cfg{}

s = box.schema.space.create('S', {format = {{'I', 'integer'}, {'D', 'decimal'}}, if_not_exists=true})
_ = s:create_index('ii', {if_not_exists=true})

local res, err = box.execute("INSERT INTO s VALUES (1, ?)", {decimal.new(0)})

if err ~= nil then
   print(err)
   os.exit(1)
end

Actual result: Bind value type USERDATA for parameter 1 is not supported

@kyukhin kyukhin added feature A new functionality sql labels Jan 14, 2020
@kyukhin kyukhin added this to the 2.4.1 milestone Jan 14, 2020
@kyukhin kyukhin modified the milestones: 2.4.1, 2.5.1 Mar 30, 2020
@kyukhin kyukhin added the prio1 label Apr 10, 2020
@kyukhin
Copy link
Contributor

kyukhin commented Jun 8, 2020

Blocked by #4415

@kyukhin kyukhin added blocked Not ready to be implemented and removed prio1 labels Jun 8, 2020
@kyukhin kyukhin modified the milestones: 2.5.1, 2.6.1 Jun 8, 2020
@slumber slumber self-assigned this Jun 16, 2020
@kyukhin kyukhin modified the milestones: 2.6.1, wishlist Oct 23, 2020
@ImeevMA ImeevMA assigned ImeevMA and unassigned slumber Aug 13, 2021
@ImeevMA ImeevMA added the teamL label Aug 13, 2021
@kyukhin kyukhin added the tmp label Aug 19, 2021
@kyukhin kyukhin modified the milestones: wishlist, 2.10.1 Aug 19, 2021
@kyukhin kyukhin removed blocked Not ready to be implemented tmp labels Aug 19, 2021
@ImeevMA ImeevMA added the 1sp label Aug 19, 2021
ImeevMA added a commit that referenced this issue Aug 26, 2021
After this patch, DECIMAL values can be bound like any other supported
by SQL values.

Closes #4717
ImeevMA added a commit that referenced this issue Aug 30, 2021
After this patch, DECIMAL values can be bound like any other supported
by SQL values.

Closes #4717
@kyukhin kyukhin closed this as completed in a94986f Sep 1, 2021
@igormunkin igormunkin removed the teamL label Oct 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature A new functionality sql
Projects
None yet
Development

No branches or pull requests

5 participants