NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit92e1583

committed

Don't do logical replication of TRUNCATE of zero tables

When due to publication configuration, a TRUNCATE change ends up withzero tables to be published, don't send the message out, just skip it.It's not wrong, but obviously useless overhead.

1 parente348e7a commit92e1583Copy full SHA for 92e1583

File tree

1 file changed

+10

-7

lines changed

src/backend/replication/pgoutput
- pgoutput.c

1 file changed

+10

-7

lines changed

`‎src/backend/replication/pgoutput/pgoutput.c`

Lines changed: 10 additions & 7 deletions

Original file line number	Diff line number	Diff line change
`@@ -407,13 +407,16 @@ pgoutput_truncate(LogicalDecodingContext ctx, ReorderBufferTXN txn,`
`407`	`407`	`maybe_send_schema(ctx,relation,relentry);`
`408`	`408`	`}`
`409`	`409`
`410`		`-OutputPluginPrepareWrite(ctx, true);`
`411`		`-logicalrep_write_truncate(ctx->out,`
`412`		`-nrelids,`
`413`		`-relids,`
`414`		`-change->data.truncate.cascade,`
`415`		`-change->data.truncate.restart_seqs);`
`416`		`-OutputPluginWrite(ctx, true);`
	`410`	`+if (nrelids>0)`
	`411`	`+{`
	`412`	`+OutputPluginPrepareWrite(ctx, true);`
	`413`	`+logicalrep_write_truncate(ctx->out,`
	`414`	`+nrelids,`
	`415`	`+relids,`
	`416`	`+change->data.truncate.cascade,`
	`417`	`+change->data.truncate.restart_seqs);`
	`418`	`+OutputPluginWrite(ctx, true);`
	`419`	`+}`
`417`	`420`
`418`	`421`	`MemoryContextSwitchTo(old);`
`419`	`422`	`MemoryContextReset(data->context);`

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!

Commit92e1583

File tree

1 file changed

1 file changed

`‎src/backend/replication/pgoutput/pgoutput.c`

0 commit comments