NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commite4e87a3

Amit Kapila

committed

Fix valgrind issue in commit05c8482.

Initialize other newly added variables in max_parallel_hazard_context viais_parallel_safe() because we don't check the parallel-safety of targetrelations in that function.Reported-by: Tom Lane as per buildfarmAuthor: Amit KapilaDiscussion:https://postgr.es/m/2060179.1615347455@sss.pgh.pa.us

1 parent05c8482 commite4e87a3Copy full SHA for e4e87a3

File tree

1 file changed

-3

lines changed

src/backend/optimizer/util
- clauses.c

1 file changed

-3

lines changed

`‎src/backend/optimizer/util/clauses.c`

Lines changed: 4 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -614,9 +614,10 @@ is_parallel_safe(PlannerInfo root, Node node)`
`614`	`614`	`context.max_hazard=PROPARALLEL_SAFE;`
`615`	`615`	`context.max_interesting=PROPARALLEL_RESTRICTED;`
`616`	`616`	`context.safe_param_ids=NIL;`
`617`		`-context.command_type=node!=NULL&&IsA(node,Query) ?`
`618`		`-castNode(Query,node)->commandType :CMD_UNKNOWN;`
`619`		`-context.planner_global=root->glob;`
	`617`	`+/* We don't need to evaluate target relation's parallel-safety here. */`
	`618`	`+context.target_rte=NULL;`
	`619`	`+context.command_type=CMD_UNKNOWN;`
	`620`	`+context.planner_global=NULL;`
`620`	`621`
`621`	`622`	`/*`
`622`	`623`	`* The params that refer to the same or parent query level are considered`

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!

Commite4e87a3

File tree

1 file changed

1 file changed

`‎src/backend/optimizer/util/clauses.c`

0 commit comments