upstream/postgresql
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2026-04-26 00:31:07 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

pg_plan_advice: pgindent

Browse source 
Reported-by: Lukas Fittl <lukas@fittl.com>
This commit is contained in:
Robert Haas 2026-03-26 20:10:13 -04:00
parent 30d432502b
commit 874da8b1f6
4 changed files with 14 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
contrib/pg_plan_advice/pgpa_planner.c
View file

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

12
contrib/pg_plan_advice/pgpa_planner.h
View file

@ -42,12 +42,12 @@ typedef struct pgpa_planner_info
/* /*
* If has_rtoffset is true, then rtoffset is the offset required to align * 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. * RTIs for this query level with RTIs from the final, flattened
* If has_rtoffset is false, then this subquery's range table wasn't copied, * rangetable. If has_rtoffset is false, then this subquery's range table
* or was only partially copied, into the final range table. (Note that * wasn't copied, or was only partially copied, into the final range
* we can't determine the rtoffset values until the final range table * table. (Note that we can't determine the rtoffset values until the
* actually exists; before that time, has_rtoffset will be false everywhere * final range table actually exists; before that time, has_rtoffset will
* except at the top level.) * be false everywhere except at the top level.)
*/ */
bool has_rtoffset; bool has_rtoffset;
Index rtoffset; Index rtoffset;

4
contrib/pg_plan_advice/pgpa_walker.c
View file

@ -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 * 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. * (e.g. MinMaxAgg losers), so pgpa_compute_rti_from_identifier won't work
* Instead, check directly against the do_not_scan_identifiers list. * here. Instead, check directly against the do_not_scan_identifiers list.
*/ */
if (tag == PGPA_TAG_DO_NOT_SCAN) if (tag == PGPA_TAG_DO_NOT_SCAN)
{ {

4
src/include/nodes/pathnodes.h
View file

@ -324,8 +324,8 @@ struct PlannerInfo
* If this PlannerInfo exists to consider an alternative implementation * If this PlannerInfo exists to consider an alternative implementation
* strategy for a portion of the query that could also be implemented by * strategy for a portion of the query that could also be implemented by
* some other PlannerInfo, this is the plan_name for that other * some other PlannerInfo, this is the plan_name for that other
* PlannerInfo. When we are considering the first or only alternative, * PlannerInfo. When we are considering the first or only alternative, it
* it is the same as plan_name. * is the same as plan_name.
* *
* Currently, we set this to a value other than plan_name only when * Currently, we set this to a value other than plan_name only when
* considering a MinMaxAggPath or a hashed SubPlan. * considering a MinMaxAggPath or a hashed SubPlan.