Re: [Monetdb-developers] [Monetdb-pf-checkins] pathfinder/compiler/debug logdebug.c, , 1.131, 1.132

Hi Tom, this looks like a bug fix to me. Shouldn't that checkin then have gone to the new Aug2009 release branch? Otherwise, this fix will surely not be in the next releases (Aug2009, Aug2009-SP1, Aug2009-SP2) ... Let me know, in case you need advice or help with back-porting this one to the Aug2009 release branch (using CVS commands, only, to prevent conflicts), and with checkin in future bug fixes to the release branch. Thanks! Stefan On Tue, Aug 04, 2009 at 08:50:10AM +0000, Tom Schreiber wrote:

Update of /cvsroot/monetdb/pathfinder/compiler/debug In directory 23jxhf1.ch3.sourceforge.com:/tmp/cvs-serv12470/compiler/debug

Modified Files: logdebug.c Log Message: -- Fixed a small bug in the dot output for algebra plans: planheader-port-id's and their references were unaligned (causing warnings from the dot executable).

U logdebug.c Index: logdebug.c =================================================================== RCS file: /cvsroot/monetdb/pathfinder/compiler/debug/logdebug.c,v retrieving revision 1.131 retrieving revision 1.132 diff -u -d -r1.131 -r1.132 --- logdebug.c 31 Jul 2009 13:06:21 -0000 1.131 +++ logdebug.c 4 Aug 2009 08:50:07 -0000 1.132 @@ -2621,7 +2621,7 @@ for (unsigned int j = 0; j < clsize (root->sem.ser_rel.items); j++) fprintf (f, "%s %s ", (j ? "|" : ""), - j, + j + 1, PFcol_str (clat (root->sem.ser_rel.items, j))); fprintf (f, "}}}\"];\n" "subgraph clusterPlan%i {\n", id);

------------------------------------------------------------------------------ Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day trial. Simplify your report design, integration and deployment - and focus on what you do best, core application coding. Discover what's new with Crystal Reports now. http://p.sf.net/sfu/bobj-july _______________________________________________ Monetdb-pf-checkins mailing list Monetdb-pf-checkins@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-pf-checkins

-- | Dr. Stefan Manegold | mailto:Stefan.Manegold@cwi.nl | | CWI, P.O.Box 94079 | http://www.cwi.nl/~manegold/ | | 1090 GB Amsterdam | Tel.: +31 (20) 592-4212 | | The Netherlands | Fax : +31 (20) 592-4312 |