Do not reindex concurrently on PG < v16 (#71)

README.md

@@ -76,7 +76,7 @@ $ docker run --name pgauto -it \
 	<NAME_OF_THE_PGAUTOUPGRADE_IMAGE>
 ```
 
-### Skip reindexing
+### Reindexing
 
 By default, all databases are reindexed after the migration, which can take some time if they are large.
 To skip reindexing, set the environment variable `PGAUTO_REINDEX` to `no`, for example:
@@ -89,6 +89,9 @@ $ docker run --name pgauto -it \
 	<NAME_OF_THE_PGAUTOUPGRADE_IMAGE>
 ```
 
+> [!WARNING]
+> PG v15 and below do not support reindexing system tables in a database concurrently. This means, when we run start the indexing operation, database locks are placed which do not allow for any modifications as long as the task is running. We recommend using PG v16 or v17 where this is not an issue.
+
 # For Developers
 
 ## Building the image

pgautoupgrade-postupgrade.sh

@@ -58,19 +58,12 @@ if [ "x${PGAUTO_REINDEX}" != "xno" ]; then
         echo "------------------------"
         echo "Reindexing the databases"
         echo "------------------------"
-
-        # For each database, reindex it
-        for DATABASE in ${DB_LIST}; do
-        	echo "-------------------------------"
-        	echo "Starting reindex of ${DATABASE}"
-        	echo "-------------------------------"
-
-        	echo 'REINDEX DATABASE CONCURRENTLY' | psql --username="${POSTGRES_USER}" -t --csv "${DATABASE}"
-
-        	echo "-------------------------------"
-        	echo "Finished reindex of ${DATABASE}"
-        	echo "-------------------------------"
-        done
+
+		if [[ "$PGTARGET" -le 15 ]]; then
+			reindexdb --all --username="${POSTGRES_USER}"
+		else
+			reindexdb --all --concurrently --username="${POSTGRES_USER}"
+		fi
 
         echo "-------------------------------"
         echo "End of reindexing the databases"