This is an automated email from the ASF dual-hosted git repository.

yjhjstz pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/cloudberry.git

commit 59675ae9f4b778d6082bf32eb0f36026584b78c3
Author: QingMa <[email protected]>
AuthorDate: Thu May 9 16:22:43 2024 +0800

    Expose several functions for extensions (#17380)
    
    This commit includes the following modifications:
    - Expose `get_common_group_tles()` and `choose_grouping_locus()`
    - `CustomScan->custom_private` should not be touched by
    `experssion_tree_walker()`.
    
    Co-authored-by: QingMa <[email protected]>
---
 src/backend/cdb/cdbtargeteddispatch.c | 2 ++
 src/backend/optimizer/util/walkers.c  | 2 --
 src/include/cdb/cdbgroupingpaths.h    | 1 -
 3 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/backend/cdb/cdbtargeteddispatch.c 
b/src/backend/cdb/cdbtargeteddispatch.c
index 9cd0b3e30c..f12e5203f1 100644
--- a/src/backend/cdb/cdbtargeteddispatch.c
+++ b/src/backend/cdb/cdbtargeteddispatch.c
@@ -516,6 +516,8 @@ DirectDispatchUpdateContentIdsFromPlan(PlannerInfo *root, 
Plan *plan)
                        break;
                case T_SplitUpdate:
                        break;
+               case T_CustomScan:
+                       break;
                default:
                        elog(ERROR, "unknown plan node %d", nodeTag(plan));
                        break;
diff --git a/src/backend/optimizer/util/walkers.c 
b/src/backend/optimizer/util/walkers.c
index ee86cfd8cd..1b4d35df0e 100644
--- a/src/backend/optimizer/util/walkers.c
+++ b/src/backend/optimizer/util/walkers.c
@@ -259,8 +259,6 @@ plan_tree_walker(Node *node,
                                return true;
                        if (walker(((CustomScan *) node)->custom_exprs, 
context))
                                return true;
-                       if (walker(((CustomScan *) node)->custom_private, 
context))
-                               return true;
                        if (walker(((CustomScan *) node)->custom_scan_tlist, 
context))
                                return true;
                        break;
diff --git a/src/include/cdb/cdbgroupingpaths.h 
b/src/include/cdb/cdbgroupingpaths.h
index 6e26d11ccb..7d4067e140 100644
--- a/src/include/cdb/cdbgroupingpaths.h
+++ b/src/include/cdb/cdbgroupingpaths.h
@@ -53,7 +53,6 @@ extern Path *cdb_prepare_path_for_hashed_agg(PlannerInfo 
*root,
                                                                                
         PathTarget *target,
                                                                                
         List *groupClause,
                                                                                
         List *rollups);
-
 extern List *get_common_group_tles(PathTarget *target,
                                                                   List 
*groupClause,
                                                                   List 
*rollups);


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to