Rafi Shamim

Another functionality we could explore adding to this option is to avoid querying the `attidentity` and `attgenerated` columns. See https://github.com/cockroachdb/cockroach/issues/48050 and https://github.com/cockroachdb/cockroach/issues/48541

The CI here is failing because https://github.com/brianc/node-postgres/pull/3110 needs to be merged (cc @brianc) But this PR is ready for review.

this is totally unrelated to your PR, but while we are adding one system database upgrade migration, do you think we could fix https://github.com/cockroachdb/cockroach/issues/113296 as well?

Until we resolve this, one option you could consider is to use the `default_transaction_priority` session variable. It controls the default priority for all transactions created in the current session, so...

> Unfortunately I don't think so as it looks like CRDB does not support the WITH (bucket_count.. in the CREATE INDEX (@rafiss am I right? I saw that through experimenting...

Nice find on that discrepancy. In this case, it's unlikely that we would make a change to the syntax, so this will stay as it is.

CockroachDB issue: https://github.com/cockroachdb/cockroach/issues/41943

I wonder if we should change the `smallserial` alias in CockroachDB so it uses a different function that generates a smaller int? Like implement a `unique_smallrowid()` function, perhaps. I don't...

> But, if you queried a JSONB column, you'd get a text value back that was the TEXT encoding of the JSONB. ? It does seem that way. This is...

Linking this other investigation that landed on the same issue: https://github.com/prisma/prisma/issues/11317#issuecomment-1065414426