Rope

Rope copied to clipboard

Removed unterminated single quote

NerdOfCode

Array support

1

Currently Rope fails to parse the OID of an array and so returns nil. - It would probably be good if Rope logged a warning that an unsupported column type...

swizzlr

Reconnect to database

1

Rope's behaviour isn't well defined right now but it should keep attempting to reconnect to the database indefinitely (I think, or with a backoff). We should also expose the health...

swizzlr

Instead of returning `[String:Any?]` let's return `[String:PostgresType]`: ```swift enum PostgresType { case text(String?), case int(Int?), case other(Any?) } ``` Nil in the associated value means null.

johanneserhardt

Dropping in a placeholder here. I'll implement one in a project and we can backport it upstream. >One thread restriction is that no two threads attempt to manipulate the same...

swizzlr

By convention, these are "postgres" and "" (no password/empty string).

swizzlr

This is a dangerous rather than sensible default. The user should have to explicitly specify the database.

swizzlr

Edge-cases / failures are not fully covered, yet:

skreutzberger

As discussed with @johanneserhardt I proposed that Rope have one more dependency: `pq` This is how we do it: ### C module map we have `Cpq` module providing module maps...

swizzlr

johanneserhardt