Details
-
Bug
-
Status: Open
-
Critical
-
Resolution: Unresolved
-
None
-
None
-
None
Description
Take CreateTableProcedure as an example, in executeFromState()
case CREATE_TABLE_PRE_OPERATION: // Verify if we can create the table boolean exists = !prepareCreate(env); releaseSyncLatch(); if (exists) { assert isFailed() : "the delete should have an exception here"; return Flow.NO_MORE_STATE; }
The following assertion:
assert isFailed() : "the delete should have an exception here";
If I get the idea behind "the delete should have an exception here" correctly, it is to make sure that when setting the state to FAILED, the exception must be set. (or must call setFailure()). But the assertion only check isFailed() but no "!hasException()"