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); replref--; return MAL_SUCCEED; }
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