pg_plan_advice: pgindent

Reported-by: Lukas Fittl <lukas@fittl.com>

diff --git a/contrib/pg_plan_advice/pgpa_planner.c b/contrib/pg_plan_advice/pgpa_planner.c
index afa9587a72561fc7d45f3138f26c4b9aeb49213f..b25f62b2e87c88d23a24bdce401e46ace0ab4183 100644 (file)
--- a/contrib/pg_plan_advice/pgpa_planner.c
+++ b/contrib/pg_plan_advice/pgpa_planner.c
@@ -2100,10 +2100,10 @@ pgpa_compute_rt_offsets(pgpa_planner_state *pps, PlannedStmt *pstmt)
                                if (rtinfo->dummy)
                                {
                                        /*
-                                        * It will not be possible to make any effective use of the
-                                        * sj_unique_rels list in this case, and it also won't be
-                                        * important to do so. So just throw the list away to avoid
-                                        * confusing pgpa_plan_walker.
+                                        * It will not be possible to make any effective use of
+                                        * the sj_unique_rels list in this case, and it also won't
+                                        * be important to do so. So just throw the list away to
+                                        * avoid confusing pgpa_plan_walker.
                                         */
                                        proot->sj_unique_rels = NIL;
                                        break;

diff --git a/contrib/pg_plan_advice/pgpa_planner.h b/contrib/pg_plan_advice/pgpa_planner.h
index 93fda2055b23192977f00c7669685a4706db295a..32808b265942e7ae603d1832591ff4f70f3676cf 100644 (file)
--- a/contrib/pg_plan_advice/pgpa_planner.h
+++ b/contrib/pg_plan_advice/pgpa_planner.h
@@ -42,12 +42,12 @@ typedef struct pgpa_planner_info
 
        /*
         * If has_rtoffset is true, then rtoffset is the offset required to align
-        * RTIs for this query level with RTIs from the final, flattened rangetable.
-        * If has_rtoffset is false, then this subquery's range table wasn't copied,
-        * or was only partially copied, into the final range table. (Note that
-        * we can't determine the rtoffset values until the final range table
-        * actually exists; before that time, has_rtoffset will be false everywhere
-        * except at the top level.)
+        * RTIs for this query level with RTIs from the final, flattened
+        * rangetable. If has_rtoffset is false, then this subquery's range table
+        * wasn't copied, or was only partially copied, into the final range
+        * table. (Note that we can't determine the rtoffset values until the
+        * final range table actually exists; before that time, has_rtoffset will
+        * be false everywhere except at the top level.)
         */
        bool            has_rtoffset;
        Index           rtoffset;

diff --git a/contrib/pg_plan_advice/pgpa_walker.c b/contrib/pg_plan_advice/pgpa_walker.c
index 0a4512d49210b4edd039807aafca04d3b1b3c04f..e32684d20750a6b777bbcc27526395c28d7086f4 100644 (file)
--- a/contrib/pg_plan_advice/pgpa_walker.c
+++ b/contrib/pg_plan_advice/pgpa_walker.c
@@ -722,8 +722,8 @@ pgpa_walker_would_advise(pgpa_plan_walker_context *walker,
 
        /*
         * DO_NOT_SCAN advice targets rels that may not be in the flat range table
-        * (e.g. MinMaxAgg losers), so we can't use pgpa_compute_rti_from_identifier.
-        * Instead, check directly against the do_not_scan_identifiers list.
+        * (e.g. MinMaxAgg losers), so pgpa_compute_rti_from_identifier won't work
+        * here. Instead, check directly against the do_not_scan_identifiers list.
         */
        if (tag == PGPA_TAG_DO_NOT_SCAN)
        {

diff --git a/src/include/nodes/pathnodes.h b/src/include/nodes/pathnodes.h
index 96ec32272624da1e738a71ae7f380fcb2bec25bf..7947d83d584b075e371f973915b5a544ed094be0 100644 (file)
--- a/src/include/nodes/pathnodes.h
+++ b/src/include/nodes/pathnodes.h
@@ -324,8 +324,8 @@ struct PlannerInfo
         * If this PlannerInfo exists to consider an alternative implementation
         * strategy for a portion of the query that could also be implemented by
         * some other PlannerInfo, this is the plan_name for that other
-        * PlannerInfo. When we are considering the first or only alternative,
-        * it is the same as plan_name.
+        * PlannerInfo. When we are considering the first or only alternative, it
+        * is the same as plan_name.
         *
         * Currently, we set this to a value other than plan_name only when
         * considering a MinMaxAggPath or a hashed SubPlan.