Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

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

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit9801bae

Browse files
committed
Lock all relations referred to in updatable views
Even views considered "simple" enough to be automatically updatable mayhave mulitple relations involved (eg: in a where clause). We need tomake sure and lock those relations when rewriting the query.Back-patch to 9.3 where updatable views were added.Pointed out by Andres, patch thanks to Dean Rasheed.
1 parent97b7b95 commit9801bae

File tree

1 file changed

+15
-0
lines changed

1 file changed

+15
-0
lines changed

‎src/backend/rewrite/rewriteHandler.c

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2757,6 +2757,21 @@ rewriteTargetView(Query *parsetree, Relation view)
27572757

27582758
heap_close(base_rel,NoLock);
27592759

2760+
/*
2761+
* If the view query contains any sublink subqueries then we need to also
2762+
* acquire locks on any relations they refer to. We know that there won't
2763+
* be any subqueries in the range table or CTEs, so we can skip those, as
2764+
* in AcquireRewriteLocks.
2765+
*/
2766+
if (viewquery->hasSubLinks)
2767+
{
2768+
acquireLocksOnSubLinks_contextcontext;
2769+
2770+
context.for_execute= true;
2771+
query_tree_walker(viewquery,acquireLocksOnSubLinks,&context,
2772+
QTW_IGNORE_RC_SUBQUERIES);
2773+
}
2774+
27602775
/*
27612776
* Create a new target RTE describing the base relation, and add it to the
27622777
* outer query's rangetable. (What's happening in the next few steps is

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp