On Tue, 17 May 2022 at 04:21, Peter Eisentraut
<peter.eisentr...@enterprisedb.com> wrote:
>      Oid        *collations;     /* cache keys */

> Is this a mistake, or could this be clarified?

Yeah, must be a copy-pasto.  I'll fix it with the attached after beta1
is tagged.

David
diff --git a/src/include/nodes/plannodes.h b/src/include/nodes/plannodes.h
index e319e83bd8..0ea9a22dfb 100644
--- a/src/include/nodes/plannodes.h
+++ b/src/include/nodes/plannodes.h
@@ -824,8 +824,9 @@ typedef struct Memoize
        int                     numKeys;                /* size of the two 
arrays below */
 
        Oid                *hashOperators;      /* hash operators for each key 
*/
-       Oid                *collations;         /* cache keys */
-       List       *param_exprs;        /* exprs containing parameters */
+       Oid                *collations;         /* collations for each key */
+       List       *param_exprs;        /* cache keys in the form of exprs 
containing
+                                                                * parameters */
        bool            singlerow;              /* true if the cache entry 
should be marked as
                                                                 * complete 
after we store the first tuple in
                                                                 * it. */

Reply via email to