NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitc621467

Etsuro Fujita

committed

Update comment about set_join_pathlist_hook().

The comment introduced by commite7cb7ee was a bit too terse, whichcould lead to extensions doing different things within the hook functionthan we intend to allow. Extend the comment to explain what they can dowithin the hook function.Back-patch to all supported branches.In passing, I rephrased a nearby comment that I recently added to theback branches.Reviewed by David Rowley and Andrei Lepikhov.Discussion:https://postgr.es/m/CAPmGK15SBPA1nr3Aqsdm%2BYyS-ay0Ayo2BRYQ8_A2To9eLqwopQ%40mail.gmail.com

1 parent5cfba1a commitc621467Copy full SHA for c621467

File tree

1 file changed

-1

lines changed

src/backend/optimizer/path
- joinpath.c

1 file changed

-1

lines changed

`‎src/backend/optimizer/path/joinpath.c‎`

Lines changed: 4 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -333,7 +333,10 @@ add_paths_to_joinrel(PlannerInfo *root,`
`333`	`333`	`jointype,&extra);`
`334`	`334`
`335`	`335`	`/*`
`336`		`- * 6. Finally, give extensions a chance to manipulate the path list.`
	`336`	`+ * 6. Finally, give extensions a chance to manipulate the path list. They`
	`337`	`+ * could add new paths (such as CustomPaths) by calling add_path(), or`
	`338`	`+ * add_partial_path() if parallel aware. They could also delete or modify`
	`339`	`+ * paths added by the core code.`
`337`	`340`	`*/`
`338`	`341`	`if (set_join_pathlist_hook)`
`339`	`342`	`set_join_pathlist_hook(root,joinrel,outerrel,innerrel,`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitc621467

File tree

1 file changed

1 file changed

`‎src/backend/optimizer/path/joinpath.c‎`

0 commit comments