mirror of https://github.com/citusdata/citus.git
Merge pull request #4817 from citusdata/marcocitus/fix-warning
Remove unnecessary AtEOXact_Files callpull/4819/head
commit
7d8d5cad98
|
@ -318,28 +318,8 @@ CoordinatedTransactionCallback(XactEvent event, void *arg)
|
||||||
/* stop propagating notices from workers, we know the query is failed */
|
/* stop propagating notices from workers, we know the query is failed */
|
||||||
DisableWorkerMessagePropagation();
|
DisableWorkerMessagePropagation();
|
||||||
|
|
||||||
/*
|
|
||||||
* FIXME: Add warning for the COORD_TRANS_COMMITTED case. That
|
|
||||||
* can be reached if this backend fails after the
|
|
||||||
* XACT_EVENT_PRE_COMMIT state.
|
|
||||||
*/
|
|
||||||
|
|
||||||
/*
|
|
||||||
* Call other parts of citus that need to integrate into
|
|
||||||
* transaction management. Do so before doing other work, so the
|
|
||||||
* callbacks still can perform work if needed.
|
|
||||||
*/
|
|
||||||
{
|
|
||||||
/*
|
|
||||||
* On Windows it's not possible to delete a file before you've closed all
|
|
||||||
* handles to it (rmdir will return success but not take effect). Since
|
|
||||||
* we're in an ABORT handler it's very likely that not all handles have
|
|
||||||
* been closed; force them closed here before running
|
|
||||||
* RemoveIntermediateResultsDirectory.
|
|
||||||
*/
|
|
||||||
AtEOXact_Files(false);
|
|
||||||
RemoveIntermediateResultsDirectory();
|
RemoveIntermediateResultsDirectory();
|
||||||
}
|
|
||||||
ResetShardPlacementTransactionState();
|
ResetShardPlacementTransactionState();
|
||||||
|
|
||||||
/* handles both already prepared and open transactions */
|
/* handles both already prepared and open transactions */
|
||||||
|
|
Loading…
Reference in New Issue