Skip to content

Commit 407d045

Browse files
committed
fix: address PR review feedback
- Restore postgres_dba_pgvers_17plus and _13plus version checks - Revert VACUUM to uppercase (SQL command) - Revert 'Postgres' terminology (preferred over PostgreSQL) - Revert DO & UNDO to uppercase (migration operations)
1 parent 90bcda6 commit 407d045

File tree

6 files changed

+14
-6
lines changed

6 files changed

+14
-6
lines changed

init/generate.sh

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,10 @@ cat > "$WARMUP" <<- VersCheck
1616
select 1/0;
1717
\endif
1818
19+
select current_setting('server_version_num')::integer >= 170000 as postgres_dba_pgvers_17plus \gset
20+
21+
select current_setting('server_version_num')::integer >= 130000 as postgres_dba_pgvers_13plus \gset
22+
1923
select current_setting('server_version_num')::integer >= 100000 as postgres_dba_pgvers_10plus \gset
2024
\if :postgres_dba_pgvers_10plus
2125
\set postgres_dba_last_wal_receive_lsn pg_last_wal_receive_lsn

sql/i5_indexes_migration.sql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
--Cleanup unused and redundant indexes – do and undo migration DDL
1+
--Cleanup unused and redundant indexes – DO & UNDO migration DDL
22

33
-- Use it to generate a database migration (e.g. RoR's db:migrate or Sqitch)
44
-- to drop unused and redundant indexes.

sql/t1_tuning.sql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
--PostgreSQL parameters tuning
1+
--Postgres parameters tuning
22

33
-- For Postgres versions older than 10, copy/paste the part
44
-- below the last "\else" (scroll down)

sql/v2_autovacuum_progress_and_queue.sql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
--Vacuum progress and autovacuum queue
1+
--VACUUM progress and autovacuum queue
22

33
-- Based on: https://gitlab.com/snippets/1889668
44

start.psql

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,17 +15,17 @@
1515
\echo ' i2 – Redundant indexes'
1616
\echo ' i3 – Foreign keys with missing or bad indexes'
1717
\echo ' i4 – Invalid indexes'
18-
\echo ' i5 – Cleanup unused and redundant indexes – do and undo migration DDL'
18+
\echo ' i5 – Cleanup unused and redundant indexes – DO & UNDO migration DDL'
1919
\echo ' l1 – Lock trees (lightweight)'
2020
\echo ' l2 – Lock trees, detailed (based on pg_blocking_pids())'
2121
\echo ' p1 – [EXP] Alignment padding: how many bytes can be saved if columns are reordered?'
2222
\echo ' r1 – Create user with random password (interactive)'
2323
\echo ' r2 – Alter user with random password (interactive)'
2424
\echo ' s1 – Slowest queries, by total time (requires pg_stat_statements)'
2525
\echo ' s2 – Slowest queries report (requires pg_stat_statements)'
26-
\echo ' t1 – PostgreSQL parameters tuning'
26+
\echo ' t1 – Postgres parameters tuning'
2727
\echo ' v1 – Vacuum: current activity'
28-
\echo ' v2 – Vacuum progress and autovacuum queue'
28+
\echo ' v2 – VACUUM progress and autovacuum queue'
2929
\echo ' q – Quit'
3030
\echo
3131
\echo Type your choice and press <Enter>:

warmup.psql

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,10 @@
44
select 1/0;
55
\endif
66

7+
select current_setting('server_version_num')::integer >= 170000 as postgres_dba_pgvers_17plus \gset
8+
9+
select current_setting('server_version_num')::integer >= 130000 as postgres_dba_pgvers_13plus \gset
10+
711
select current_setting('server_version_num')::integer >= 100000 as postgres_dba_pgvers_10plus \gset
812
\if :postgres_dba_pgvers_10plus
913
\set postgres_dba_last_wal_receive_lsn pg_last_wal_receive_lsn

0 commit comments

Comments
 (0)