Chris Browne cbbrowne at lists.slony.info
Thu Feb 28 11:23:03 PST 2008
Update of /home/cvsd/slony1/slony1-engine/src/backend
In directory main.slony.info:/tmp/cvs-serv7592

Modified Files:
	slony1_funcs.sql 
Log Message:
Fixes needed to get "testpartition" to work on HEAD. The issue was that
with the new handling of triggers, it is no longer appropriate to run
the "restore table" function, as that function was only apropos for 1.2
and earlier, and has become deprecated in HEAD.


Index: slony1_funcs.sql
===================================================================
RCS file: /home/cvsd/slony1/slony1-engine/src/backend/slony1_funcs.sql,v
retrieving revision 1.132
retrieving revision 1.133
diff -C2 -d -r1.132 -r1.133
*** slony1_funcs.sql	25 Feb 2008 15:42:32 -0000	1.132
--- slony1_funcs.sql	28 Feb 2008 19:23:01 -0000	1.133
***************
*** 4054,4057 ****
--- 4054,4064 ----
  'alterTableRestore (tab_id)
  
+ Note: This function only functions properly when used on pre-2.0
+ systems being converted into 2.0 form.  In Slony-I 2.0, the trigger
+ handling has changed substantially, such that:
+ 
+ - There are *two* triggers on each table, created at "creation time", and
+ - There is no need to run "restore" as part of the DDL/EXECUTE SCRIPT process.
+ 
  Restores table tab_id from being replicated.
  
***************
*** 5782,5787 ****
  	v_idxname := p_idxname;
     end if;
!    perform @NAMESPACE at .setAddTable_int(p_set_id, p_tab_id, v_fqname, v_idxname, p_comment);
!    return @NAMESPACE at .alterTableRestore(p_tab_id);
  end
  ' language plpgsql;
--- 5789,5793 ----
  	v_idxname := p_idxname;
     end if;
!    return @NAMESPACE at .setAddTable_int(p_set_id, p_tab_id, v_fqname, v_idxname, p_comment);
  end
  ' language plpgsql;



More information about the Slony1-commit mailing list