observe() on table with String key?

[siuying]

The observe interface takes a Int id:

public func observe<T : Sqlable>(change: Sqlable.SqliteDatabase.Change? = default, on: T.Type, id: Int? = default, doThis: (id: Int) throws -> Void) -> String

Problem is I have to use String key for my model (some of the integer is bigger than 32bit and will not work on older devices).

[ulrikdamm]

I'll take a look at string id support. Right now there's a few places that assumes integer ids.

[siuying]

Do you think it's okay if I just replace Int with SqlValue? I can investigate this and prepare a PR, the return value will still use the rowid returned.

[ulrikdamm]

It's not that simple, because when you get an update callback from Sqlite, you only get the rowid. We'd have to make a SQL call to get the string value, so that we can compare them.
One solution I was thinking about was to get the rowid when registering an observer on a string primary key, but that wouldn't work for insert-observers, and it would have to fail if the observed id didn't exist.