Commit7441d66

Etsuro Fujita

authored and

pull[bot]

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 parenta98a4d7 commit7441d66Copy full SHA for 7441d66

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!

Commit7441d66

File tree

1 file changed

1 file changed

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

0 commit comments