提交 8c9c0576 编写于 作者: H Heikki Linnakangas

Fix obsolete comment in cdb_build_distribution_pathkeys().

It returns a simple list of PathKeys, not a list of lists. The code was
changed in the 8.3-era merge of equivalence classes already, but we
neglected the comment.
上级 50f2e3bb
......@@ -189,10 +189,7 @@ cdbpathlocus_equal(CdbPathLocus a, CdbPathLocus b)
* cdb_build_distribution_pathkeys
* Build canonicalized pathkeys list for given columns of rel.
*
* Returns a List, of length 'nattrs': each of its members is
* a List of one or more PathKey nodes. The returned List
* might contain duplicate entries: this occurs when the
* corresponding columns are constrained to be equal.
* Returns a List of PathKeys, of length 'nattrs'.
*
* The caller receives ownership of the returned List, freshly
* palloc'ed in the caller's context. The members of the returned
......@@ -216,12 +213,9 @@ cdb_build_distribution_pathkeys(PlannerInfo *root,
/* Find or create a Var node that references the specified column. */
Var *expr = find_indexkey_var(root, rel, attrs[i]);
/*
* Find or create a pathkey.
*/
/* Find or create a pathkey. */
cpathkey = cdb_make_pathkey_for_expr(root, (Node *) expr, eq);
/* Append to list of pathkeys. */
retval = lappend(retval, cpathkey);
}
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册