Robin11/19/2021, 6:34 PM
and just want to make sure that the SpiceDB devs ( @User @User ) are aware of it... While I read through your watch crdb code https://github.com/authzed/spicedb/blob/a75cd6484c313c71dfd98072dadf0a4fffaed4d0/internal/datastore/crdb/watch.go#L29 and the crdb docs https://www.cockroachlabs.com/docs/v21.2/changefeed-for#considerations
- Since you use a pgx.Pool underneath, are you sure that the connection gets closed after the context to Watch stops watching, and that the pg connect is not just released back into the pool leaving the connection in an unusable state for new normal SQL queries?
It is therefore critical that you close the connection, otherwise the application will be blocked forever on Rows.Close().