Update of /cvsroot/monetdb/sql/src/backends/monet5
In directory
23jxhf1.ch3.sourceforge.com:/tmp/cvs-serv12755/sql/src/backends/monet5
Modified Files:
Tag: Feb2009
sql_gencode.mx
Log Message:
fixing the leak reported in
[ 2540169 ] Memory leak and crash with UNION ALL
https://sourceforge.net/tracker/index.php?func=detail&aid=2540169&group_id=56967&atid=482468
by making sure that we use the BAT returned by bat.append since
~~~~~~~
2008/08/09 - mlkersten: MonetDB5/src/modules/kernel/bat5.mx,1.37
The access modes of the kernel BAT update operations were not aligned.
This caused the SQL front-end to inject setWriteMode calls before each
append, and possibly trimming it again with an optimizer.
The BAT insert, append, delete, and replace functions now always return
the first argument, which my be a copy with the proper access properties.
This elicits the flow. The side effectness can be checked with the
isUpdateFunction.
The majority of files involve the now visible temporary in the output.
~~~~~~~
Index: sql_gencode.mx
===================================================================
RCS file: /cvsroot/monetdb/sql/src/backends/monet5/sql_gencode.mx,v
retrieving revision 1.312
retrieving revision 1.312.2.1
diff -u -d -r1.312 -r1.312.2.1
--- sql_gencode.mx 27 Jan 2009 15:20:15 -0000 1.312
+++ sql_gencode.mx 30 Jan 2009 23:52:16 -0000 1.312.2.1
@@ -1411,7 +1411,7 @@
q = pushArgument(mb, q, l);
q = pushArgument(mb, q, r);
q = pushBit(mb, q, TRUE);
- s->nr = l;
+ s->nr = getDestVar(q);
} break;
case st_update_col:
case st_append_col:{
------------------------------------------------------------------------------
This SF.net email is sponsored by:
SourcForge Community
SourceForge wants to tell your story.
http://p.sf.net/sfu/sf-spreadtheword
_______________________________________________
Monetdb-sql-checkins mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/monetdb-sql-checkins
