comment typo in postgres_fdw/postgres_fdw.c

Started by Ian Lawrence Barwickabout 12 years ago2 messages
#1Ian Lawrence Barwick
barwick@gmail.com
1 attachment(s)

Presumably "classifyConditions", not "classifyClauses", is meant.
Patch attached.

Regards

Ian Barwick

Attachments:

postgresql_fdw-typo.difftext/plain; charset=US-ASCII; name=postgresql_fdw-typo.diffDownload
diff --git a/contrib/postgres_fdw/postgres_fdw.c b/contrib/postgres_fdw/postgres_fdw.c
index 246a3a9..46ea032 100644
--- a/contrib/postgres_fdw/postgres_fdw.c
+++ b/contrib/postgres_fdw/postgres_fdw.c
@@ -723,7 +723,7 @@ postgresGetForeignPlan(PlannerInfo *root,
 	/*
 	 * Separate the scan_clauses into those that can be executed remotely and
 	 * those that can't.  baserestrictinfo clauses that were previously
-	 * determined to be safe or unsafe by classifyClauses are shown in
+	 * determined to be safe or unsafe by classifyConditions are shown in
 	 * fpinfo->remote_conds and fpinfo->local_conds.  Anything else in the
 	 * scan_clauses list should be a join clause that was found safe by
 	 * postgresGetForeignPaths.
#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Ian Lawrence Barwick (#1)
Re: comment typo in postgres_fdw/postgres_fdw.c

Ian Lawrence Barwick <barwick@gmail.com> writes:

Presumably "classifyConditions", not "classifyClauses", is meant.

Hm, I think this got missed in a bout of renaming. Will apply, thanks.

regards, tom lane

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers