NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commita3f17b9

committed

Wake up launcher when enabling a subscription

Otherwise one would have to wait up to DEFAULT_NAPTIME_PER_CYCLE untilthe subscription worker is considered for starting.There is a small race condition: If one enables a subscription rightafter disabling it, the launcher might not have registered the stoppingwhen receiving the wakeup signal for the re-enabling. The start willthen not happen right away but after the full cycle time.Author: Kyotaro HORIGUCHI <horiguchi.kyotaro@lab.ntt.co.jp>

1 parentef0ba57 commita3f17b9Copy full SHA for a3f17b9

File tree

1 file changed

-0

lines changed

src/backend/commands
- subscriptioncmds.c

1 file changed

-0

lines changed

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

Lines changed: 3 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -644,6 +644,9 @@ AlterSubscription(AlterSubscriptionStmt *stmt)`
`644`	`644`	`BoolGetDatum(enabled);`
`645`	`645`	`replaces[Anum_pg_subscription_subenabled-1]= true;`
`646`	`646`
	`647`	`+if (enabled)`
	`648`	`+ApplyLauncherWakeupAtCommit();`
	`649`	`+`
`647`	`650`	`update_tuple= true;`
`648`	`651`	`break;`
`649`	`652`	`}`

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!

Commita3f17b9

File tree

1 file changed

1 file changed

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

0 commit comments