Skip to content

Fix replace count test#9262

Merged
timsehn merged 1 commit intomainfrom
tim/fix-replace-count
May 28, 2025
Merged

Fix replace count test#9262
timsehn merged 1 commit intomainfrom
tim/fix-replace-count

Conversation

@timsehn
Copy link
Copy Markdown
Contributor

@timsehn timsehn commented May 28, 2025

Claude insists that this behavior matches MySQL. It consulted the docs and ran tests against MySQL.

I can confirm the tested behavior matches MariaDB which I have installed:

MariaDB [(none)]> CREATE DATABASE test_replace;
ERROR 1007 (HY000): Can't create database 'test_replace'; database exists
MariaDB [(none)]> DROP DATABASE IF EXISTS
    ->       test_replace;
Query OK, 1 row affected (0.057 sec)

MariaDB [(none)]> USE test_replace;
ERROR 1049 (42000): Unknown database 'test_replace'
MariaDB [(none)]> CREATE DATABASE test_replace;
Query OK, 1 row affected (0.003 sec)

MariaDB [(none)]> USE test_replace;
Database changed
MariaDB [test_replace]> CREATE TABLE test(pk BIGINT
    ->       PRIMARY KEY, v BIGINT);
Query OK, 0 rows affected (0.030 sec)

MariaDB [test_replace]> REPLACE INTO test
    ->       VALUES (1, 1);
Query OK, 1 row affected (0.001 sec)

MariaDB [test_replace]> REPLACE INTO test VALUES (1,
    ->       2);
Query OK, 2 rows affected (0.001 sec)

@timsehn timsehn merged commit ca6e388 into main May 28, 2025
24 checks passed
@timsehn
Copy link
Copy Markdown
Contributor Author

timsehn commented May 28, 2025

Zach review this over Discord.

@tbantle22 tbantle22 deleted the tim/fix-replace-count branch May 28, 2025 21:35
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

Successfully merging this pull request may close these issues.

1 participant