jakajancar / pgc4d

A full-featured PostgreSQL Client for Deno
MIT License
21 stars 3 forks source link

Feature request: Add a pool #3

Open grantcarthew opened 4 years ago

grantcarthew commented 4 years ago

Have a look here:

https://github.com/brianc/node-postgres/tree/master/packages/pg-pool

Whilst you are there, check out the other packages within the node-postgres project. The pg-connection string is written in TypeScript. Some of the other packages could be migrated. Many hours of design, build, and debugging have gone into those packages.

jakajancar commented 4 years ago

TBH I'm not convinced of the benefits of a database driver including its own pool vs. just using a generic one.

Fwiw, I use olpo and it works OK.

grantcarthew commented 4 years ago

OK, here is a question then. How often have you used a database driver without using a pool? If the answer is never or a low number, why make it harder to connect to your database?

A large number of database drivers include a pool. It is a small addition that will be appreciated by almost all users of the driver.