Update of /cvsroot/monetdb/pathfinder/compiler/algebra/prop
In directory 23jxhf1.ch3.sourceforge.com:/tmp/cvs-serv4226/compiler/algebra/prop

Modified Files:
      Tag: M5XQ
        prop_const.c 
Log Message:
propagated changes of Friday Jun 12 2009 - Monday Jun 15 2009
from the development trunk to the M5XQ branch

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2009/06/12 - tsheyar: compiler/algebra/prop/prop_const.c,1.53
-- Replaced aggregate operators count, min, max, avg, sum, prod, seqty1,
   and all in the algebra by a single aggregate operator ``aggr''
   that can handle multiple aggregates. The aggregate entries
   are of kind count, min, max, avg, sum, prod, seqty1, all, and dist.

-- Added new aggregate kind ``dist'' that allows to represent group by
   columns that functionally depend on the partitioning criterion
   in the result of the grouping aggregate.

-- Added rewrite that merges aggregates.

-- Added rewrite that removes superfluous aggregates.

-- Added rewrite that pushes a rank operator through an aggregate.

-- Extended the XML import to cope with the old
   as well as the new representation of aggregates.
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~


U prop_const.c
Index: prop_const.c
===================================================================
RCS file: /cvsroot/monetdb/pathfinder/compiler/algebra/prop/prop_const.c,v
retrieving revision 1.51.2.1
retrieving revision 1.51.2.2
diff -u -d -r1.51.2.1 -r1.51.2.2
--- prop_const.c        7 May 2009 14:44:18 -0000       1.51.2.1
+++ prop_const.c        15 Jun 2009 12:45:46 -0000      1.51.2.2
@@ -538,20 +538,25 @@
                     PFprop_const_val (L(n)->prop, n->sem.binary.col1));
             break;
 
-        case la_avg:
-        case la_max:
-        case la_min:
-            /* if column 'col' is constant the result 'res' will be as well */
-            if (PFprop_const (L(n)->prop, n->sem.aggr.col))
-                PFprop_mark_const (
-                        n->prop,
-                        n->sem.aggr.res,
-                        PFprop_const_val (L(n)->prop, n->sem.aggr.col));
-        case la_sum:
-        case la_prod:
-        case la_count:
-        case la_seqty1:
-        case la_all:
+        case la_aggr:
+            for (unsigned int i = 0; i < n->sem.aggr.count; i++)
+                if (PFprop_const (L(n)->prop, n->sem.aggr.aggr[i].col))
+                    switch (n->sem.aggr.aggr[i].kind) {
+                        case alg_aggr_dist:
+                        case alg_aggr_min:
+                        case alg_aggr_max:
+                        case alg_aggr_avg:
+                        case alg_aggr_all:
+                            PFprop_mark_const (
+                                    n->prop,
+                                    n->sem.aggr.aggr[i].res,
+                                    PFprop_const_val (L(n)->prop,
+                                                      
n->sem.aggr.aggr[i].col));
+                            break;
+
+                        default:
+                            break;
+                    }
             if (n->sem.aggr.part &&
                 PFprop_const (L(n)->prop, n->sem.aggr.part))
                 PFprop_mark_const (


------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
_______________________________________________
Monetdb-pf-checkins mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/monetdb-pf-checkins

Reply via email to