external
/
citus
mirror of https://github.com/citusdata/citus.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #3991 from citusdata/fix/remove-level-assert 

Remove executor/planner level asserts in abort handler
pull/3976/head
Marco Slot 2020-07-14 22:43:24 +02:00 committed by GitHub
parent bc011a6286 5fbb925df1
commit e09860e9e3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/backend/distributed/transaction/transaction_management.c
View File

@ -316,19 +316,11 @@ CoordinatedTransactionCallback(XactEvent event, void *arg)
ResetGlobalVariables(); ResetGlobalVariables();
/* /*
* Getting here without ExecutorLevel 0 is a bug, however it is such a big * We reset these mainly for posterity. The only way we would normally
* problem that will persist between reuse of the backend we still assign 0 in * get here with ExecutorLevel or PlannerLevel > 0 is during a fatal
* production deploys, but during development and tests we want to crash. * error when the process is about to end.
*/ */
Assert(ExecutorLevel == 0);
ExecutorLevel = 0; ExecutorLevel = 0;
/*
* Getting here without PlannerLevel 0 is a bug, however it is such a big
* problem that will persist between reuse of the backend we still assign 0 in
* production deploys, but during development and tests we want to crash.
*/
Assert(PlannerLevel == 0);
PlannerLevel = 0; PlannerLevel = 0;
/* /*