when to disconnect and when to end a pg client or pool

First, from the pg documentation*:

const { Pool } = require('pg')const pool = new Pool()// the pool with emit an error on behalf of any idle clients// it contains if a backend error or network partition happenspool.on('error', (err, client) => {  console.error('Unexpected error on idle client', err) // your callback here  process.exit(-1)})// promise - checkout a clientpool.connect()  .then(client => {    return client.query('SELECT * FROM users WHERE id = $1', [1]) // your query string here      .then(res => {        client.release()        console.log(res.rows[0]) // your callback here      })      .catch(e => {        client.release()        console.log(err.stack) // your callback here      })  })

This code/construct is suficient/made to get your pool working, providing the your thing here things. If you shut down your application, the connection will hang normaly, since the pool is created well, exactly not to hang, even if it does provides a manual way of hanging,see last section of the article.Also look at the previous red section which says "You must always return the client..." to accept

• the mandatory client.release() instruction
• before accesing argument.
• you scope/closure client within your callbacks.

Then, from the pg.client documentation*:

Plain text query with a promise

const { Client } = require('pg').Clientconst client = new Client()client.connect()client.query('SELECT NOW()') // your query string here  .then(result => console.log(result)) // your callback here  .catch(e => console.error(e.stack)) // your callback here  .then(() => client.end())

seems to me the clearest syntax:

• you end the client whatever the results.
• you access the result before ending the client.
• you don´t scope/closure the client within your callbacks

It is this sort of oposition between the two syntaxes that may be confusing at first sight, but there is no magic in there, it is implementation construction syntax.Focus on your callbacks and queries, not on those constructs, just pick up the most elegant for your eyes and feed it with your code.

*I added the comments // your xxx here for clarity

You shouldn't disconnect the pool on every query, connection pool is supposed to be used to have "hot" connections.

I usually have a global connection on startup and the pool connection close on (if) application stop; you just have to release the connection from pool every time the query ends, as you already do, and use the same pool also in the signup function.

Sometimes I need to preserve connections, I use a wrapper to the query function that checks if the connection is active or not before perform the query, but it's just an optimization.

In case you don't want to manage open/close connections/pool or release, you could try https://github.com/vitaly-t/pg-promise, it manage all that stuff silently and it works well.

The documentation over node-postgres's github says:

pro tip: unless you need to run a transaction (which requires a single client for multiple queries) or you have some other edge case like streaming rows or using a cursor you should almost always just use pool.query. Its easy, it does the right thing ™️, and wont ever forget to return clients back to the pool after the query is done.

So for non-transactional query, calling below code is enough.

var pool = new Pool()pool.query('select username from user WHERE username= $1',[username], function(err, res) {  console.log(res.rows[0].username)})