NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitb21aed3

committed

Acquire appropriate locks when rewriting during RMV.

Since the query has not been freshly parsed when executing REFRESHMATERIALIZED VIEW, locks must be explicitly taken before rewrite.Backpatch to 9.3.Andres Freund

1 parentb2cd72c commitb21aed3Copy full SHA for b21aed3

File tree

1 file changed

-2

lines changed

src/backend/commands
- matview.c

1 file changed

-2

lines changed

`‎src/backend/commands/matview.c`

Lines changed: 5 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -242,9 +242,12 @@ refresh_matview_datafill(DestReceiver dest, Query query,`
`242`	`242`	`List*rewritten;`
`243`	`243`	`PlannedStmt*plan;`
`244`	`244`	`QueryDesc*queryDesc;`
	`245`	`+Query*copied_query;`
`245`	`246`
`246`		`-/* Rewrite, copying the given Query to make sure it's not changed */`
`247`		`-rewritten=QueryRewrite((Query*)copyObject(query));`
	`247`	`+/* Lock and rewrite, using a copy to preserve the original query. */`
	`248`	`+copied_query=copyObject(query);`
	`249`	`+AcquireRewriteLocks(copied_query, false);`
	`250`	`+rewritten=QueryRewrite(copied_query);`
`248`	`251`
`249`	`252`	`/* SELECT should never rewrite to more or less than one SELECT query */`
`250`	`253`	`if (list_length(rewritten)!=1)`

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!

Commitb21aed3

File tree

1 file changed

1 file changed

`‎src/backend/commands/matview.c`

0 commit comments