NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit6de6d96

committed

Invalidate caches after cranking up a parallel worker transaction.

Starting a parallel worker transaction changes our notion of which XIDsare in-progress or committed, and our notion of the current commandcounter ID. Therefore, our view of these caches prior to startingthis transaction may no longer valid. Defend against that by clearingthem.This fixes a bug in commit924bcf4.

1 parentd07fea5 commit6de6d96Copy full SHA for 6de6d96

File tree

1 file changed

-0

lines changed

src/backend/access/transam
- parallel.c

1 file changed

-0

lines changed

`‎src/backend/access/transam/parallel.c`

Lines changed: 7 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,7 @@`
`28`	`28`	`#include"tcop/tcopprot.h"`
`29`	`29`	`#include"utils/combocid.h"`
`30`	`30`	`#include"utils/guc.h"`
	`31`	`+#include"utils/inval.h"`
`31`	`32`	`#include"utils/memutils.h"`
`32`	`33`	`#include"utils/resowner.h"`
`33`	`34`	`#include"utils/snapmgr.h"`
`@@ -928,6 +929,12 @@ ParallelWorkerMain(Datum main_arg)`
`928`	`929`	`Assert(asnapspace!=NULL);`
`929`	`930`	`PushActiveSnapshot(RestoreSnapshot(asnapspace));`
`930`	`931`
	`932`	`+/*`
	`933`	`+ * We've changed which tuples we can see, and must therefore invalidate`
	`934`	`+ * system caches.`
	`935`	`+ */`
	`936`	`+InvalidateSystemCaches();`
	`937`	`+`
`931`	`938`	`/* Restore user ID and security context. */`
`932`	`939`	`SetUserIdAndSecContext(fps->current_user_id,fps->sec_context);`
`933`	`940`

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!

Commit6de6d96

File tree

1 file changed

1 file changed

`‎src/backend/access/transam/parallel.c`

0 commit comments