Repository navigation
chore: additional path quailification - #2592
staaldraad wants to merge 3 commits into
Conversation
Fully qualify all paths before a search_path switch. Standardise the way of doing this across all scripts.
Package version diff: none (aarch64-linux, x86_64-linux) |
There was a problem hiding this comment.
May also need migrations.
20250218031949_pgsodium_mask_role.sqlstill createsmask_rolewithSET search_path TO ''.create extension pgsodiumgetspg_catalog, pg_tempfromafter-create.sql, while projects migrated through that file keep''.- similar for
pgmqwith the requested changes
Two small new migrations with the same mask_role/drop_queue definitions would remove the drift.
pinaraf
left a comment
There was a problem hiding this comment.
The last flag of set_config, "local", makes it pointless to manually restore the search_path. So this can be greatly simplified...
Other than that, specifying a full path everywhere is the way to go...
Wrong: I was not aware these files are included in others, making this comment moot. |
PostgreSQL Extension Dependency Analysis: PR #2592
SummaryNo extensions had dependencies with MAJOR version updates. Full Analysis ResultsPostgreSQL 15 Extension DependenciesPostgreSQL 17 Extension DependenciesOrioleDB 17 Extension Dependencies |
PostgreSQL Package Dependency Analysis: PR #2592
SummaryNo packages had MAJOR version updates. Full Analysis ResultsPostgreSQL 15 Dependency ChangesExtracting PostgreSQL 15 dependencies...
Runtime Closure Size
Raw Dependency ClosurePostgreSQL 17 Dependency ChangesExtracting PostgreSQL 17 dependencies...
Runtime Closure Size
Raw Dependency ClosureOrioleDB 17 Dependency ChangesExtracting OrioleDB 17 depen...[Comment body truncated] |
Fully qualify all paths before a search_path switch.
Standardise the way of doing this across all scripts.