Commitdff2f63

committed

Enforce REPEATABLE READ since currently we export snap only once in xact

There is also no distribute predicate locking support, so SERIALIZABLE is notthe way to go too.

1 parentd545692 commitdff2f63Copy full SHA for dff2f63

File tree

-0

lines changed

-0

lines changed

Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -460,6 +460,10 @@ begin_remote_xact(ConnCacheEntry *entry)`
`460`	`460`	`elog(DEBUG3,"starting remote transaction on connection %p",`
`461`	`461`	`entry->conn);`
`462`	`462`
	`463`	`+if (UseGlobalSnapshots&& (!IsolationUsesXactSnapshot()\|\|`
	`464`	`+IsolationIsSerializable()))`
	`465`	`+elog(ERROR,"Global snapshots support only REPEATABLE READ");`
	`466`	`+`
`463`	`467`	`if (IsolationIsSerializable())`
`464`	`468`	`sql="START TRANSACTION ISOLATION LEVEL SERIALIZABLE";`
`465`	`469`	`else`

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -12,6 +12,7 @@`
`12`	`12`	`log_checkpoints = true`
`13`	`13`	`postgres_fdw.use_global_snapshots = on`
`14`	`14`	`track_global_snapshots = on`
	`15`	`+default_transaction_isolation = 'REPEATABLE READ'`
`15`	`16`	`));`
`16`	`17`	`$master->start;`
`17`	`18`

Lines changed: 2 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -12,6 +12,7 @@`
`12`	`12`	`postgres_fdw.use_global_snapshots = on`
`13`	`13`	`global_snapshot_defer_time = 15`
`14`	`14`	`track_global_snapshots = on`
	`15`	`+ default_transaction_isolation = 'REPEATABLE READ'`
`15`	`16`	`));`
`16`	`17`	`$shard1->start;`
`17`	`18`
`@@ -22,6 +23,7 @@`
`22`	`23`	`postgres_fdw.use_global_snapshots = on`
`23`	`24`	`global_snapshot_defer_time = 15`
`24`	`25`	`track_global_snapshots = on`
	`26`	`+default_transaction_isolation = 'REPEATABLE READ'`
`25`	`27`	`));`
`26`	`28`	`$shard2->start;`
`27`	`29`

Comments

(0)