get rid of {Push/Pop}OverrideSearchPath #7145
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PushOverrideSearchPath suffers from a vulnerability issue reported by
CVE-2023-2454. Postgres fix this by replacing it with set_config_option, see [1].
It is recommended that out-of-tree code should also update such code though
the "override" mechanism remains for compatibility.
The postgres master branch(i.e. PG 17) recently removed
PushOverrideSearchPath() and PopOverrideSearchPath(), see [2]. So this should
also ease the process when citus decide to support Postgres v17.
This patch also contains some trivial typo fix.
[1]: postgres commit: 681d9e4621aac0a9c71364b6f54f00f6d8c4337f
[2]: postgres commit: 7c5c4e1c0396b0617a6f9b659dd7375fb0bfb9dc