Update of /cvsroot/monetdb/sql/src/server
In directory sc8-pr-cvs16.sourceforge.net:/tmp/cvs-serv10667

Modified Files:
        sql_parser.mx 
Log Message:
Distinct should also be used with scalar expressions.

No tests were broken with this addition and the syntax error on skyserver query 
logs is gone.
However, a test to cover the semantics and syntax of this change need to be 
added...


U sql_parser.mx
Index: sql_parser.mx
===================================================================
RCS file: /cvsroot/monetdb/sql/src/server/sql_parser.mx,v
retrieving revision 1.280
retrieving revision 1.281
diff -u -d -r1.280 -r1.281
--- sql_parser.mx       1 Apr 2008 17:04:58 -0000       1.280
+++ sql_parser.mx       9 May 2008 14:00:54 -0000       1.281
@@ -3537,6 +3537,12 @@
                  append_int(l, TRUE);
                  append_symbol(l, _symbol_create_list(SQL_COLUMN, $4));
                  $$ = _symbol_create_list( SQL_AGGR, l ); }
+ |  AGGR '(' DISTINCT scalar_exp ')'
+               { dlist *l = L();
+                 append_string(l, $1);
+                 append_int(l, FALSE);
+                 append_symbol(l, $4);
+                 $$ = _symbol_create_list( SQL_AGGR, l ); }
  |  AGGR '(' ALL scalar_exp ')'
                { dlist *l = L();
                  append_string(l, $1);


-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
_______________________________________________
Monetdb-sql-checkins mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/monetdb-sql-checkins

Reply via email to