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

Problem with the test and REPLACE #3

Open
agershun opened this issue Jan 1, 2016 · 1 comment
Open

Problem with the test and REPLACE #3

agershun opened this issue Jan 1, 2016 · 1 comment

Comments

@agershun
Copy link
Member

agershun commented Jan 1, 2016

This is a the part of the test file

Last lines are:

REPLACE INTO t1 VALUES(4, 'replace (new)')

query IT rowsort
SELECT x, y FROM t1 WHERE x=4
----
4
replace (new)

But here there are no any lines with x=4. How REPLACE INTO should work here? Should it insert new value or just skip?

@mathiasrw
Copy link
Member

Well.... its a good question. It seem a bit counter intuitive to insert when replace is requested.

To comply with the other DBs I recommend that REPLACE would be implemented as a short version of INSERT OR REPLACE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants