Text primary key not supported in v0.1.6?

[a-h]

With this input:

create virtual table text_pk using vec0(
  pk text not null primary key,
  embedding float[3],
);

insert or replace into text_pk (pk, embedding) values ('a', '[1, 2, 3]');
insert or replace into text_pk (pk, embedding) values ('a', '[4, 5, 6]');

select rowid, pk, vec_to_json(embedding) from text_pk;

I expected one row with pk 'a' and embedding '[4, 5, 6]', but got:

+-------+----+------------------------------+
| rowid | pk | vec_to_json(embedding)       |
+-------+----+------------------------------+
| 1     | a  | [1.000000,2.000000,3.000000] |
+-------+----+------------------------------+
| 2     | a  | [4.000000,5.000000,6.000000] |
+-------+----+------------------------------+

Is this a bug?

[asg017]

This is a bug, an error should be thrown because not null is not supported in vec0 constructors yet.

Removing that should enforce primary keys:

create virtual table text_pk using vec0(
  pk text primary key,
  embedding float[3],
);

insert or replace into text_pk (pk, embedding) values ('a', '[1, 2, 3]');
insert or replace into text_pk (pk, embedding) values ('a', '[4, 5, 6]'); -- Runtime error: UNIQUE constraint failed on text_pk primary key

Though also keep in mind, insert or replace into is quite supported in vec0 virtual tables yet, see #127

So there are two bugs, one for incorrect parsing of primary keys when not null is defined, and a second one for upserts. I'll try to fix both this week!