postgres.git / summary / log / commit / refs

commit    fdd79d8992d600d38eecfab568b27f6d04e4dfb3
Author:   David Rowley <drowley@postgresql.org>
Date:     Mon Aug 07 06:16:46 2023 +0000
First released in: 17.0

    Fix misleading comment in paraminfo_get_equal_hashops
    
    The comment mistakenly claimed the code was checking PlaceHolderVars for
    volatile functions when the code was actually checking lateral vars.
    
    Update the comment to reflect the reality of the code.
    
    Author: Richard Guo
    Discussion: https://postgr.es/m/CAMbWs48HZGZOV85g0fx8z1qDx6NNKHexJPT2FCnKnZhxBWkd-A@mail.gmail.com


 src/backend/optimizer/path/joinpath.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/backend/optimizer/path/joinpath.c b/src/backend/optimizer/path/joinpath.c
index 4b58936fa4c..c1d478d2eb2 100644
--- a/src/backend/optimizer/path/joinpath.c
+++ b/src/backend/optimizer/path/joinpath.c
@@ -529,7 +529,7 @@ paraminfo_get_equal_hashops(PlannerInfo *root, ParamPathInfo *param_info,
 		Node	   *expr = (Node *) lfirst(lc);
 		TypeCacheEntry *typentry;
 
-		/* Reject if there are any volatile functions in PHVs */
+		/* Reject if there are any volatile functions in lateral vars */
 		if (contain_volatile_functions(expr))
 		{
 			list_free(*operators);


[parent: 6c00d2c9d4c5]