Skip to content

Commit 968709c

Browse files
committed
Revert parallel query NULL checks that caused CI timeout
The NULL checks for exec_state_call_stack added in commits 99d9d2f and 31c1136 were causing general JDBC tests to timeout. These checks were intended to fix parallel query crashes but inadvertently broke the normal execution path. This reverts those specific changes while keeping the important fixes: - 1c80f61: Fix INSERT EXEC snapshot error with AFTER triggers - df12e05: Fix INSERT EXEC INSTEAD OF triggers on view targets - 7d020a7: Fix C90 failure Parallel query tests may still fail (separate issue), but general JDBC tests should now pass without timeout.
1 parent 7d020a7 commit 968709c

File tree

1 file changed

+0
-8
lines changed

1 file changed

+0
-8
lines changed

contrib/babelfishpg_tsql/src/iterative_exec.c

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -951,10 +951,6 @@ is_part_of_pltsql_trycatch_block(PLtsql_execstate *estate)
951951
PLExecStateCallStack *cur;
952952
int level = 0;
953953

954-
/* Safety check for parallel workers where exec_state_call_stack may be NULL */
955-
if (exec_state_call_stack == NULL)
956-
return false;
957-
958954
Assert(estate == exec_state_call_stack->estate);
959955
cur = exec_state_call_stack;
960956
while (cur != NULL)
@@ -992,10 +988,6 @@ is_part_of_pltsql_trigger(PLtsql_execstate *estate)
992988
{
993989
PLExecStateCallStack *cur;
994990

995-
/* Safety check for parallel workers where exec_state_call_stack may be NULL */
996-
if (exec_state_call_stack == NULL)
997-
return false;
998-
999991
Assert(estate == exec_state_call_stack->estate);
1000992
cur = exec_state_call_stack;
1001993
while (cur != NULL)

0 commit comments

Comments
 (0)