Romulo Goncalves wrote:
Update of /cvsroot/monetdb/MonetDB5/src/optimizer In directory sc8-pr-cvs16.sourceforge.net:/tmp/cvs-serv18911
Modified Files: opt_replicator.mx Log Message: If you stop you should free the structure that you allocked in REPstart...
Compilation works and also the replicator test...
Index: opt_replicator.mx
RCS file: /cvsroot/monetdb/MonetDB5/src/optimizer/opt_replicator.mx,v retrieving revision 1.23 retrieving revision 1.24 diff -u -d -r1.23 -r1.24 --- opt_replicator.mx 29 Feb 2008 07:29:21 -0000 1.23 +++ opt_replicator.mx 29 Feb 2008 13:13:20 -0000 1.24 @@ -132,7 +132,7 @@ opt_export str REPsemijoin(MalBlkPtr mb, MalStkPtr stk, InstrPtr pci); opt_export str REPdump(int *ret); opt_export str REPstart(MalBlkPtr mb, MalStkPtr stk, InstrPtr pci); -opt_export str REPstop(void); +opt_export str REPstop(MalStkPtr stk); opt_export str REPsetRetain(int *ret, sht *p); opt_export str REPsetReuse(int *ret, sht *p); opt_export str REPsetCache(int *ret, sht *p); @@ -242,8 +242,10 @@ }
-str REPstop(void) +str REPstop(MalStkPtr s) {
- if( s->blk->replica )
GDKfree(((ReplicaRec*)s->blk->replica)->vmap);}
In principle yes, but in this case I don't think so. The vmap is the map to replicated variables, that can be used next time the same query is executed, which is after rep.stop. Pls, revert Milena
This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ Monetdb-checkins mailing list Monetdb-checkins@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-checkins