Comment on top of RangeVarGetRelidExtended in namespace.c mentioning RangeVarGetRelid

Started by Michael Paquieralmost 8 years ago3 messages
#1Michael Paquier
michael@paquier.xyz
1 attachment(s)

Hi all,

While reviewing the NOWAIT patch for VACUUM (well, SKIP_LOCKED per these
day's trends), I bumped into $subject.

Attached is a one-liner.
Thanks,
--
Michael

Attachments:

namespace-c-typo.patchtext/plain; charset=us-asciiDownload
diff --git a/src/backend/catalog/namespace.c b/src/backend/catalog/namespace.c
index 65e271a8d1..52dd400b96 100644
--- a/src/backend/catalog/namespace.c
+++ b/src/backend/catalog/namespace.c
@@ -202,7 +202,7 @@ static bool MatchNamedCall(HeapTuple proctup, int nargs, List *argnames,
 
 
 /*
- * RangeVarGetRelid
+ * RangeVarGetRelidExtended
  *		Given a RangeVar describing an existing relation,
  *		select the proper namespace and look up the relation OID.
  *
#2Stephen Frost
sfrost@snowman.net
In reply to: Michael Paquier (#1)
Re: Comment on top of RangeVarGetRelidExtended in namespace.c mentioning RangeVarGetRelid

Greetings Michael,

* Michael Paquier (michael@paquier.xyz) wrote:

While reviewing the NOWAIT patch for VACUUM (well, SKIP_LOCKED per these
day's trends), I bumped into $subject.

Attached is a one-liner.

Thanks, fix pushed.

Stephen

#3Michael Paquier
michael@paquier.xyz
In reply to: Stephen Frost (#2)
Re: Comment on top of RangeVarGetRelidExtended in namespace.c mentioning RangeVarGetRelid

On Tue, Mar 06, 2018 at 11:37:15PM -0500, Stephen Frost wrote:

Thanks, fix pushed.

Thanks, Stephen.
--
Michael