Commita6a399a

committed

[PGPRO-7614] Do not free hash_seq_search scans of parent transaction

Tags: pg_variables, atx

1 parentff47670 commita6a399aCopy full SHA for a6a399a

File tree

+65

-0

lines changed

+65

-0

lines changed

Lines changed: 21 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -266,6 +266,27 @@ ERROR: unrecognized package "vars"`
`266`	`266`	`SELECT pgv_get('vars', 'int1', null::int);`
`267`	`267`	`ERROR: unrecognized package "vars"`
`268`	`268`	`ROLLBACK;`
	`269`	`+--`
	`270`	`+--`
	`271`	`+-- Do not free hash_seq_search scans of parent transaction.`
	`272`	`+--`
	`273`	`+BEGIN;`
	`274`	`+SELECT pgv_insert('test', 'x', row (1::int, 2::int), false);`
	`275`	`+ pgv_insert`
	`276`	`+------------`
	`277`	`+`
	`278`	`+(1 row)`
	`279`	`+`
	`280`	`+DECLARE r1_cur CURSOR FOR SELECT pgv_select('test', 'x');`
	`281`	`+FETCH 1 IN r1_cur;`
	`282`	`+ pgv_select`
	`283`	`+------------`
	`284`	`+ (1,2)`
	`285`	`+(1 row)`
	`286`	`+`
	`287`	`+BEGIN AUTONOMOUS;`
	`288`	`+ROLLBACK;`
	`289`	`+ROLLBACK;`
`269`	`290`	`select pgv_free();`
`270`	`291`	`pgv_free`
`271`	`292`	`----------`

Lines changed: 25 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -296,6 +296,31 @@ WARNING: there is no transaction in progress`
`296`	`296`	`1`
`297`	`297`	`(1 row)`
`298`	`298`
	`299`	`+ROLLBACK;`
	`300`	`+WARNING: there is no transaction in progress`
	`301`	`+--`
	`302`	`+--`
	`303`	`+-- Do not free hash_seq_search scans of parent transaction.`
	`304`	`+--`
	`305`	`+BEGIN;`
	`306`	`+SELECT pgv_insert('test', 'x', row (1::int, 2::int), false);`
	`307`	`+ pgv_insert`
	`308`	`+------------`
	`309`	`+`
	`310`	`+(1 row)`
	`311`	`+`
	`312`	`+DECLARE r1_cur CURSOR FOR SELECT pgv_select('test', 'x');`
	`313`	`+FETCH 1 IN r1_cur;`
	`314`	`+ pgv_select`
	`315`	`+------------`
	`316`	`+ (1,2)`
	`317`	`+(1 row)`
	`318`	`+`
	`319`	`+BEGIN AUTONOMOUS;`
	`320`	`+ERROR: syntax error at or near "AUTONOMOUS"`
	`321`	`+LINE 1: BEGIN AUTONOMOUS;`
	`322`	`+ ^`
	`323`	`+ROLLBACK;`
`299`	`324`	`ROLLBACK;`
`300`	`325`	`WARNING: there is no transaction in progress`
`301`	`326`	`select pgv_free();`

Lines changed: 8 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -2951,7 +2951,11 @@ freeStatsLists(void)`
`2951`	`2951`	`{`
`2952`	`2952`	`VariableStatEntryentry= (VariableStatEntry)lfirst(cell);`
`2953`	`2953`
	`2954`	`+#ifdefPGPRO_EE`
	`2955`	`+hash_seq_term_all_levels(entry->status);`
	`2956`	`+#else`
`2954`	`2957`	`hash_seq_term(entry->status);`
	`2958`	`+#endif`
`2955`	`2959`	`}`
`2956`	`2960`
`2957`	`2961`	`variables_stats=NIL;`
`@@ -2960,7 +2964,11 @@ freeStatsLists(void)`
`2960`	`2964`	`{`
`2961`	`2965`	`PackageStatEntryentry= (PackageStatEntry)lfirst(cell);`
`2962`	`2966`
	`2967`	`+#ifdefPGPRO_EE`
	`2968`	`+hash_seq_term_all_levels(entry->status);`
	`2969`	`+#else`
`2963`	`2970`	`hash_seq_term(entry->status);`
	`2971`	`+#endif`
`2964`	`2972`	`}`
`2965`	`2973`
`2966`	`2974`	`packages_stats=NIL;`

Lines changed: 11 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -131,5 +131,16 @@ BEGIN;`
`131`	`131`	`-- ERROR: unrecognized package "vars"`
`132`	`132`	`SELECT pgv_get('vars','int1',null::int);`
`133`	`133`	`ROLLBACK;`
	`134`	`+--`
	`135`	`+--`
	`136`	`+-- Do not free hash_seq_search scans of parent transaction.`
	`137`	`+--`
	`138`	`+BEGIN;`
	`139`	`+SELECT pgv_insert('test','x', row (1::int,2::int), false);`
	`140`	`+DECLARE r1_cur CURSOR FORSELECT pgv_select('test','x');`
	`141`	`+FETCH1IN r1_cur;`
	`142`	`+BEGIN AUTONOMOUS;`
	`143`	`+ROLLBACK;`
	`144`	`+ROLLBACK;`
`134`	`145`
`135`	`146`	`select pgv_free();`

Comments

(0)