aboutsummaryrefslogtreecommitdiff
path: root/src/backend/parser/parse_clause.c
diff options
context:
space:
mode:
Diffstat (limited to 'src/backend/parser/parse_clause.c')
-rw-r--r--src/backend/parser/parse_clause.c9
1 files changed, 5 insertions, 4 deletions
diff --git a/src/backend/parser/parse_clause.c b/src/backend/parser/parse_clause.c
index 654dce6755e..8d90b5098a1 100644
--- a/src/backend/parser/parse_clause.c
+++ b/src/backend/parser/parse_clause.c
@@ -339,10 +339,11 @@ transformJoinUsingClause(ParseState *pstate,
/*
* We cheat a little bit here by building an untransformed operator tree
- * whose leaves are the already-transformed Vars. This is OK because
- * transformExpr() won't complain about already-transformed subnodes.
- * However, this does mean that we have to mark the columns as requiring
- * SELECT privilege for ourselves; transformExpr() won't do it.
+ * whose leaves are the already-transformed Vars. This requires collusion
+ * from transformExpr(), which normally could be expected to complain
+ * about already-transformed subnodes. However, this does mean that we
+ * have to mark the columns as requiring SELECT privilege for ourselves;
+ * transformExpr() won't do it.
*/
forboth(lvars, leftVars, rvars, rightVars)
{