NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitf5d2871

committed

Reimplement nullification of walsender timestamp

Make the value null only at pg_stat_activity-output time, as suggestedby Tom Lane, instead of messing with the internal state. This shouldappease buildfarm members with force_parallel_mode=regress, which arerunning parallel queries on logical replication walsenders.The fact that walsenders can run parallel queries should perhaps bestudied more carefully, but for the moment let's get rid of the redblots in buildfarm.Backpatch to pg10, like the previous commit.Discussion:https://postgr.es/m/30804.1578438763@sss.pgh.pa.us

1 parent913bbd8 commitf5d2871Copy full SHA for f5d2871

File tree

2 files changed

-8

lines changed

src/backend
- access/transam
  - xact.c
- utils/adt
  - pgstatfuncs.c

2 files changed

-8

lines changed

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

Lines changed: 0 additions & 7 deletions

Original file line number	Diff line number	Diff line change
`@@ -816,13 +816,6 @@ GetCurrentTransactionStopTimestamp(void)`
`816`	`816`	`void`
`817`	`817`	`SetCurrentStatementStartTimestamp(void)`
`818`	`818`	`{`
`819`		`-/*`
`820`		`- * Skip if on a walsender; this is not needed, and it confuses monitoring`
`821`		`- * if we publish non-NULL values.`
`822`		`- */`
`823`		`-if (am_walsender)`
`824`		`-return;`
`825`		`-`
`826`	`819`	`if (!IsParallelWorker())`
`827`	`820`	`stmtStartTimestamp=GetCurrentTimestamp();`
`828`	`821`	`else`

`‎src/backend/utils/adt/pgstatfuncs.c`

Lines changed: 7 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -724,7 +724,13 @@ pg_stat_get_activity(PG_FUNCTION_ARGS)`
`724`	`724`	`else`
`725`	`725`	`nulls[7]= true;`
`726`	`726`
`727`		`-if (beentry->st_xact_start_timestamp!=0)`
	`727`	`+/*`
	`728`	`+ * Don't expose transaction time for walsenders; it confuses`
	`729`	`+ * monitoring, particularly because we don't keep the time up-to-`
	`730`	`+ * date.`
	`731`	`+ */`
	`732`	`+if (beentry->st_xact_start_timestamp!=0&&`
	`733`	`+beentry->st_backendType!=B_WAL_SENDER)`
`728`	`734`	`values[8]=TimestampTzGetDatum(beentry->st_xact_start_timestamp);`
`729`	`735`	`else`
`730`	`736`	`nulls[8]= true;`

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!

Commitf5d2871

File tree

2 files changed

2 files changed

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

`‎src/backend/utils/adt/pgstatfuncs.c`

0 commit comments