more comments
authorRobert Haas <rhaas@postgresql.org>
Tue, 24 Jun 2025 16:24:29 +0000 (12:24 -0400)
committerRobert Haas <rhaas@postgresql.org>
Tue, 24 Jun 2025 16:24:29 +0000 (12:24 -0400)
contrib/pg_plan_advice/pgpa_join.c

index 81fec871699be857930a7544fd7fe92b23c01de6..c73df0b8d739a00772c99225adfe0a0159206b9b 100644 (file)
@@ -776,6 +776,9 @@ pgpa_debug_out_join_member(StringInfo buf, pgpa_join_member *member)
                appendStringInfo(buf, "%d", member->rti);
 }
 
+/*
+ * Get a C string that corresponds to the specified join clump strategy.
+ */
 static char *
 pgpa_cstring_join_clump_strategy(pgpa_join_clump_strategy strategy)
 {
@@ -792,6 +795,9 @@ pgpa_cstring_join_clump_strategy(pgpa_join_clump_strategy strategy)
        Assert(false);
 }
 
+/*
+ * Get a C string that corresponds to the specified join strategy.
+ */
 static char *
 pgpa_cstring_join_strategy(pgpa_join_strategy strategy)
 {
@@ -814,12 +820,18 @@ pgpa_cstring_join_strategy(pgpa_join_strategy strategy)
        Assert(false);
 }
 
+/*
+ * Is this a Result node that has a child?
+ */
 static bool
 is_result_node_with_child(Plan *plan)
 {
        return IsA(plan, Result) && plan->lefttree != NULL;
 }
 
+/*
+ * Is this a Plan node whose purpose is put the data in a certain order?
+ */
 static bool
 is_sorting_plan(Plan *plan)
 {