fix: Increase the db-pool-timeout to 1 hour

CHANGELOG.md

@@ -45,6 +45,7 @@ This project adheres to [Semantic Versioning](http://semver.org/).
  - #2277, #2238, #1643, Prevent views from breaking one-to-many/many-to-one embeds when using column or FK as target - @steve-chavez
     + When using a column or FK as target for embedding(`/tbl?select=*,col-or-fk(*)`), only tables are now detected and views are not.
     + You can still use a column or an inferred FK on a view to embed a table(`/view?select=*,col-or-fk(*)`)
+ - #2317, Increase the `db-pool-timeout` to 1 hour to prevent frequent high connection latency - @steve-chavez
 
 ### Changed
 

src/PostgREST/CLI.hs

@@ -148,7 +148,7 @@ exampleConfigFile =
       |db-pool = 10
       |
       |## Time to live, in seconds, for an idle database pool connection
-      |db-pool-timeout = 10
+      |db-pool-timeout = 3600
       |
       |## Stored proc to exec immediately after auth
       |# db-pre-request = "stored_proc_name"

src/PostgREST/Config.hs

@@ -214,7 +214,7 @@ parser optPath env dbSettings =
     <*> optWithAlias (optInt "db-max-rows")
                      (optInt "max-rows")
     <*> (fromMaybe 10 <$> optInt "db-pool")
-    <*> (fromIntegral . fromMaybe 10 <$> optInt "db-pool-timeout")
+    <*> (fromIntegral . fromMaybe 3600 <$> optInt "db-pool-timeout")
     <*> (fmap toQi <$> optWithAlias (optString "db-pre-request")
                                     (optString "pre-request"))
     <*> (fromMaybe True <$> optBool "db-prepared-statements")

test/io/configs/expected/aliases.config

@@ -4,7 +4,7 @@ db-channel-enabled = true
 db-extra-search-path = "public"
 db-max-rows = 1000
 db-pool = 10
-db-pool-timeout = 10
+db-pool-timeout = 3600
 db-pre-request = "check_alias"
 db-prepared-statements = true
 db-root-spec = "open_alias"

test/io/configs/expected/boolean-numeric.config

@@ -4,7 +4,7 @@ db-channel-enabled = true
 db-extra-search-path = "public"
 db-max-rows = ""
 db-pool = 10
-db-pool-timeout = 10
+db-pool-timeout = 3600
 db-pre-request = ""
 db-prepared-statements = false
 db-root-spec = ""

test/io/configs/expected/boolean-string.config

@@ -4,7 +4,7 @@ db-channel-enabled = true
 db-extra-search-path = "public"
 db-max-rows = ""
 db-pool = 10
-db-pool-timeout = 10
+db-pool-timeout = 3600
 db-pre-request = ""
 db-prepared-statements = false
 db-root-spec = ""

test/io/configs/expected/defaults.config

@@ -4,7 +4,7 @@ db-channel-enabled = true
 db-extra-search-path = "public"
 db-max-rows = ""
 db-pool = 10
-db-pool-timeout = 10
+db-pool-timeout = 3600
 db-pre-request = ""
 db-prepared-statements = true
 db-root-spec = ""

test/io/configs/expected/types.config

@@ -4,7 +4,7 @@ db-channel-enabled = true
 db-extra-search-path = "public"
 db-max-rows = ""
 db-pool = 10
-db-pool-timeout = 10
+db-pool-timeout = 3600
 db-pre-request = ""
 db-prepared-statements = true
 db-root-spec = ""