@@ -76,8 +76,6 @@ static void addToChangesStackUpperLevel(TransObject *object,
7676static void pushChangesStack (void );
7777
7878static int numOfRegVars (Package * package );
79- /* Debug function */
80- static int _numOfTransVars (Package * package );
8179
8280/* Constructors */
8381static void makePackHTAB (Package * package , bool is_trans );
@@ -906,8 +904,6 @@ remove_variable(PG_FUNCTION_ARGS)
906904 else
907905 removeObject (& variable -> transObject , TRANS_VARIABLE );
908906
909- Assert (numOfTransVars (package ) == _numOfTransVars (package ));
910-
911907 resetVariablesCache (false);
912908
913909 PG_FREE_IF_COPY (package_name , 0 );
@@ -1614,8 +1610,6 @@ createVariableInternal(Package *package, text *name, Oid typid, bool is_record,
16141610 GetPackState (package )-> trans_var_num ++ ;
16151611 GetActualState (variable )-> is_valid = true;
16161612
1617- Assert (numOfTransVars (package ) == _numOfTransVars (package ));
1618-
16191613 /* If it is necessary, put variable to changedVars */
16201614 if (is_transactional )
16211615 addToChangesStack (transObject , TRANS_VARIABLE );
@@ -1834,11 +1828,6 @@ releaseSavepoint(TransObject *object, TransObjectType type)
18341828
18351829 /* Change subxact level due to release */
18361830 GetActualState (object )-> level -- ;
1837- if (type == TRANS_PACKAGE )
1838- {
1839- Package * package = (Package * )object ;
1840- Assert (numOfTransVars (package ) == _numOfTransVars (package ));
1841- }
18421831}
18431832
18441833static void
@@ -2174,25 +2163,3 @@ _PG_fini(void)
21742163 UnregisterSubXactCallback (pgvSubTransCallback , NULL );
21752164 ExecutorEnd_hook = prev_ExecutorEnd ;
21762165}
2177-
2178- /* Get exact count of valid variables in package. For debug only. */
2179- static int
2180- _numOfTransVars (Package * package )
2181- {
2182- HASH_SEQ_STATUS vstat ;
2183- Variable * variable ;
2184- unsigned long res = 0 ;
2185-
2186- if (package -> varHashTransact )
2187- {
2188- hash_seq_init (& vstat , package -> varHashTransact );
2189- while ((variable = (Variable * ) hash_seq_search (& vstat )) != NULL )
2190- {
2191- if (GetActualState (variable )-> is_valid &&
2192- GetActualState (package )-> is_valid )
2193- res ++ ;
2194- }
2195- }
2196-
2197- return res ;
2198- }
0 commit comments