Changeset: 3fc666997e8f for MonetDB
URL: https://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=3fc666997e8f
Modified Files:
        clients/R/Tests/deps-install.timeout
        sql/backends/monet5/sql_scenario.c
        sql/server/sql_mvc.c
        sql/server/sql_mvc.h
        sql/test/BugTracker-2015/Tests/schema-trigger.Bug-3710.sql
        sql/test/BugTracker-2015/Tests/schema-trigger.Bug-3710.stable.out
        sql/test/sys-schema/Tests/systemfunctions.sql
        sql/test/sys-schema/Tests/systemfunctions.stable.out
        sql/test/sys-schema/Tests/systemfunctions.stable.out.int128
Branch: default
Log Message:

Merge with Apr2019 branch.


diffs (275 lines):

diff --git a/clients/R/Tests/deps-install.timeout 
b/clients/R/Tests/deps-install.timeout
--- a/clients/R/Tests/deps-install.timeout
+++ b/clients/R/Tests/deps-install.timeout
@@ -1,1 +1,1 @@
-2
+20
diff --git a/sql/backends/monet5/sql_scenario.c 
b/sql/backends/monet5/sql_scenario.c
--- a/sql/backends/monet5/sql_scenario.c
+++ b/sql/backends/monet5/sql_scenario.c
@@ -719,12 +719,11 @@ SQLcompile(Client cntxt, MalBlkPtr mb, M
 
        (void) mb;
        *ret = NULL;
-       msg = SQLstatementIntern(cntxt, expr, "SQLcompile", FALSE, FALSE, NULL);
-       if (msg == MAL_SUCCEED)
-               *ret = _STRDUP("SQLcompile");
-       if(*ret == NULL)
+       if ((msg = SQLstatementIntern(cntxt, expr, "SQLcompile", FALSE, FALSE, 
NULL)) != MAL_SUCCEED)
+               return msg;
+       if((*ret = _STRDUP("SQLcompile")) == NULL)
                throw(SQL,"sql.compile",SQLSTATE(HY001) MAL_MALLOC_FAIL);
-       return msg;
+       return MAL_SUCCEED;
 }
 
 /*
diff --git a/sql/server/sql_mvc.c b/sql/server/sql_mvc.c
--- a/sql/server/sql_mvc.c
+++ b/sql/server/sql_mvc.c
@@ -697,7 +697,6 @@ mvc_create(int clientid, backend_stack s
 
        m->type = Q_PARSE;
        m->pushdown = 1;
-       m->has_groupby_expressions = false;
 
        m->result_id = 0;
        m->results = NULL;
@@ -764,7 +763,6 @@ mvc_reset(mvc *m, bstream *rs, stream *w
        m->cascade_action = NULL;
        m->type = Q_PARSE;
        m->pushdown = 1;
-       m->has_groupby_expressions = false;
 
        for(i=0;i<MAXSTATS;i++)
                m->opt_stats[i] = 0;
@@ -1637,20 +1635,15 @@ stack_push_groupby_expression(mvc *sql, 
                if(res)
                        sql->topvars++;
        }
-       sql->has_groupby_expressions = true;
        return res;
 }
 
 sql_exp*
 stack_get_groupby_expression(mvc *sql, symbol *def)
 {
-       if(sql->has_groupby_expressions) {
-               for (int i = sql->topvars-1; i >= 0; i--) {
-                       if (!sql->vars[i].frame && sql->vars[i].exp && 
sql->vars[i].exp->token == def->token && symbol_cmp(sql, 
sql->vars[i].exp->sdef, def)==0) {
-                               return sql->vars[i].exp->exp;
-                       }
-               }
-       }
+       for (int i = sql->topvars-1; i >= 0; i--)
+               if (!sql->vars[i].frame && sql->vars[i].exp && 
sql->vars[i].exp->token == def->token && symbol_cmp(sql, 
sql->vars[i].exp->sdef, def)==0)
+                       return sql->vars[i].exp->exp;
        return NULL;
 }
 
diff --git a/sql/server/sql_mvc.h b/sql/server/sql_mvc.h
--- a/sql/server/sql_mvc.h
+++ b/sql/server/sql_mvc.h
@@ -107,7 +107,6 @@ typedef struct mvc {
        int argmax;
        struct symbol *sym;
        int no_mitosis;         /* run query without mitosis */
-       bool has_groupby_expressions;
 
        sqlid user_id;
        sqlid role_id;
diff --git a/sql/test/BugTracker-2015/Tests/schema-trigger.Bug-3710.sql 
b/sql/test/BugTracker-2015/Tests/schema-trigger.Bug-3710.sql
--- a/sql/test/BugTracker-2015/Tests/schema-trigger.Bug-3710.sql
+++ b/sql/test/BugTracker-2015/Tests/schema-trigger.Bug-3710.sql
@@ -1,11 +1,23 @@
 create schema marketdata;
+
 create table marketdata.quotes (i integer);
+
 CREATE TRIGGER marketdata.calc_sdate BEFORE INSERT ON marketdata.quotes
 FOR EACH ROW 
 BEGIN ATOMIC
 -- select * from marketdata.quotes limit 1;
+ update marketdata.quotes set i = i +2 where i < 2;
 END;
---syntax error, unexpected '.', expecting WHILE in: "create trigger 
marketdata.calc_sdate before insert on marketdata."
+
+insert into marketdata.quotes (i) values (1);
+select * from marketdata.quotes;
+insert into marketdata.quotes (i) values (2);
+select * from marketdata.quotes;
+insert into marketdata.quotes (i) values (1);
+select * from marketdata.quotes;
+insert into marketdata.quotes (i) values (2);
+select * from marketdata.quotes;
+
 drop trigger marketdata.calc_sdate;
 drop table marketdata.quotes;
 drop schema marketdata;
diff --git a/sql/test/BugTracker-2015/Tests/schema-trigger.Bug-3710.stable.out 
b/sql/test/BugTracker-2015/Tests/schema-trigger.Bug-3710.stable.out
--- a/sql/test/BugTracker-2015/Tests/schema-trigger.Bug-3710.stable.out
+++ b/sql/test/BugTracker-2015/Tests/schema-trigger.Bug-3710.stable.out
@@ -30,6 +30,51 @@ stdout of test 'schema-trigger.Bug-3710`
 
 #create schema marketdata;
 #create table marketdata.quotes (i integer);
+#CREATE TRIGGER marketdata.calc_sdate BEFORE INSERT ON marketdata.quotes
+#FOR EACH ROW 
+#BEGIN ATOMIC
+#-- select * from marketdata.quotes limit 1;
+# update marketdata.quotes set i = i +2 where i < 2;
+#END;
+#insert into marketdata.quotes (i) values (1);
+[ 1     ]
+#select * from marketdata.quotes;
+% marketdata.quotes # table_name
+% i # name
+% int # type
+% 1 # length
+[ 1     ]
+#insert into marketdata.quotes (i) values (2);
+[ 1     ]
+#select * from marketdata.quotes;
+% marketdata.quotes # table_name
+% i # name
+% int # type
+% 1 # length
+[ 3     ]
+[ 2     ]
+#insert into marketdata.quotes (i) values (1);
+[ 1     ]
+#select * from marketdata.quotes;
+% marketdata.quotes # table_name
+% i # name
+% int # type
+% 1 # length
+[ 3     ]
+[ 2     ]
+[ 1     ]
+#insert into marketdata.quotes (i) values (2);
+[ 1     ]
+#select * from marketdata.quotes;
+% marketdata.quotes # table_name
+% i # name
+% int # type
+% 1 # length
+[ 3     ]
+[ 2     ]
+[ 3     ]
+[ 2     ]
+#drop trigger marketdata.calc_sdate;
 #drop table marketdata.quotes;
 #drop schema marketdata;
 
diff --git a/sql/test/sys-schema/Tests/systemfunctions.stable.out 
b/sql/test/sys-schema/Tests/systemfunctions.stable.out
--- a/sql/test/sys-schema/Tests/systemfunctions.stable.out
+++ b/sql/test/sys-schema/Tests/systemfunctions.stable.out
@@ -5,46 +5,17 @@ stdout of test 'systemfunctions` in dire
 # 16:20:24 >  "mserver5" "--debug=10" "--set" "gdk_nr_threads=0" "--set" 
"mapi_open=true" "--set" "mapi_port=36066" "--set" 
"mapi_usock=/var/tmp/mtest-26153/.s.monetdb.36066" "--set" "monet_prompt=" 
"--forcemito" "--set" "mal_listing=2" 
"--dbpath=/ufs/sjoerd/Monet-candidate/var/MonetDB/mTests_sql_test" "--set" 
"mal_listing=0"
 # 16:20:24 >  
 
-# MonetDB 5 server v11.17.0 (hg id: d91089635868)
+# MonetDB 5 server v11.33.4 (hg id: e20e95cd4eff+)
 # This is an unreleased version
 # Serving database 'mTests_sql_test', using 8 threads
-# Compiled for x86_64-unknown-linux-gnu/64bit with 64bit OIDs dynamically 
linked
-# Found 15.591 GiB available main-memory.
-# Copyright (c) 1993-July 2008 CWI.
-# Copyright (c) August 2008-2015 MonetDB B.V., all rights reserved
-# Visit http://www.monetdb.org/ for further information
+# Compiled for x86_64-unknown-linux-gnu/64bit with 64bit integers
+# Found 31.305 GiB available main-memory.
+# Copyright (c) 1993 - July 2008 CWI.
+# Copyright (c) August 2008 - 2019 MonetDB B.V., all rights reserved
+# Visit https://www.monetdb.org/ for further information
 # Listening for connection requests on mapi:monetdb://madrid.ins.cwi.nl:36066/
 # Listening for UNIX domain connection requests on 
mapi:monetdb:///var/tmp/mtest-26153/.s.monetdb.36066
 # MonetDB/GIS module loaded
-# SQL catalog created, loading sql scripts once
-# loading sql script: 09_like.sql
-# loading sql script: 10_math.sql
-# loading sql script: 11_times.sql
-# loading sql script: 12_url.sql
-# loading sql script: 13_date.sql
-# loading sql script: 14_inet.sql
-# loading sql script: 15_querylog.sql
-# loading sql script: 16_tracelog.sql
-# loading sql script: 17_compress.sql
-# loading sql script: 18_dictionary.sql
-# loading sql script: 19_cluster.sql
-# loading sql script: 20_vacuum.sql
-# loading sql script: 21_dependency_functions.sql
-# loading sql script: 22_clients.sql
-# loading sql script: 23_skyserver.sql
-# loading sql script: 24_zorder.sql
-# loading sql script: 25_debug.sql
-# loading sql script: 26_sysmon.sql
-# loading sql script: 39_analytics.sql
-# loading sql script: 40_geom.sql
-# loading sql script: 40_json.sql
-# loading sql script: 41_md5sum.sql
-# loading sql script: 45_uuid.sql
-# loading sql script: 46_gsl.sql
-# loading sql script: 75_storagemodel.sql
-# loading sql script: 80_statistics.sql
-# loading sql script: 80_udf.sql
-# loading sql script: 99_system.sql
 # MonetDB/SQL module loaded
 
 
diff --git a/sql/test/sys-schema/Tests/systemfunctions.stable.out.int128 
b/sql/test/sys-schema/Tests/systemfunctions.stable.out.int128
--- a/sql/test/sys-schema/Tests/systemfunctions.stable.out.int128
+++ b/sql/test/sys-schema/Tests/systemfunctions.stable.out.int128
@@ -5,46 +5,17 @@ stdout of test 'systemfunctions` in dire
 # 16:20:24 >  "mserver5" "--debug=10" "--set" "gdk_nr_threads=0" "--set" 
"mapi_open=true" "--set" "mapi_port=36066" "--set" 
"mapi_usock=/var/tmp/mtest-26153/.s.monetdb.36066" "--set" "monet_prompt=" 
"--forcemito" "--set" "mal_listing=2" 
"--dbpath=/ufs/sjoerd/Monet-candidate/var/MonetDB/mTests_sql_test" "--set" 
"mal_listing=0"
 # 16:20:24 >  
 
-# MonetDB 5 server v11.17.0 (hg id: d91089635868)
+# MonetDB 5 server v11.33.4 (hg id: e20e95cd4eff+)
 # This is an unreleased version
 # Serving database 'mTests_sql_test', using 8 threads
-# Compiled for x86_64-unknown-linux-gnu/64bit with 64bit OIDs dynamically 
linked
-# Found 15.591 GiB available main-memory.
-# Copyright (c) 1993-July 2008 CWI.
-# Copyright (c) August 2008-2015 MonetDB B.V., all rights reserved
-# Visit http://www.monetdb.org/ for further information
+# Compiled for x86_64-unknown-linux-gnu/64bit with 128bit integers
+# Found 31.305 GiB available main-memory.
+# Copyright (c) 1993 - July 2008 CWI.
+# Copyright (c) August 2008 - 2019 MonetDB B.V., all rights reserved
+# Visit https://www.monetdb.org/ for further information
 # Listening for connection requests on mapi:monetdb://madrid.ins.cwi.nl:36066/
 # Listening for UNIX domain connection requests on 
mapi:monetdb:///var/tmp/mtest-26153/.s.monetdb.36066
 # MonetDB/GIS module loaded
-# SQL catalog created, loading sql scripts once
-# loading sql script: 09_like.sql
-# loading sql script: 10_math.sql
-# loading sql script: 11_times.sql
-# loading sql script: 12_url.sql
-# loading sql script: 13_date.sql
-# loading sql script: 14_inet.sql
-# loading sql script: 15_querylog.sql
-# loading sql script: 16_tracelog.sql
-# loading sql script: 17_compress.sql
-# loading sql script: 18_dictionary.sql
-# loading sql script: 19_cluster.sql
-# loading sql script: 20_vacuum.sql
-# loading sql script: 21_dependency_functions.sql
-# loading sql script: 22_clients.sql
-# loading sql script: 23_skyserver.sql
-# loading sql script: 24_zorder.sql
-# loading sql script: 25_debug.sql
-# loading sql script: 26_sysmon.sql
-# loading sql script: 39_analytics.sql
-# loading sql script: 40_geom.sql
-# loading sql script: 40_json.sql
-# loading sql script: 41_md5sum.sql
-# loading sql script: 45_uuid.sql
-# loading sql script: 46_gsl.sql
-# loading sql script: 75_storagemodel.sql
-# loading sql script: 80_statistics.sql
-# loading sql script: 80_udf.sql
-# loading sql script: 99_system.sql
 # MonetDB/SQL module loaded
 
 
_______________________________________________
checkin-list mailing list
[email protected]
https://www.monetdb.org/mailman/listinfo/checkin-list

Reply via email to