Changeset: c0cffd5afc2b for MonetDB
URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=c0cffd5afc2b
Modified Files:
        clients/R/MonetDB.R/R/dbi.R
        clients/R/Tests/All
        clients/R/Tests/dbi.R
        sql/backends/monet5/Tests/simpleSQL.stable.err
        sql/backends/monet5/sql_execute.c
        sql/test/BugTracker-2014/Tests/ifthenelse.Bug-3629.stable.out
        
sql/test/BugTracker-2015/Tests/crash_in_reduce_groupby.Bug-3818.stable.out
        
sql/test/BugTracker-2015/Tests/crash_in_reduce_groupby.Bug-3818.stable.out.int128
        tools/embedded/Tests/dbi.R
        tools/embedded/build-on-windows.bat
Branch: embedded
Log Message:

Fixes for testweb


diffs (156 lines):

diff --git a/clients/R/MonetDB.R/R/dbi.R b/clients/R/MonetDB.R/R/dbi.R
--- a/clients/R/MonetDB.R/R/dbi.R
+++ b/clients/R/MonetDB.R/R/dbi.R
@@ -835,12 +835,12 @@ setMethod("isIdCurrent", signature(dbObj
   .Deprecated("dbIsValid")
    dbIsValid(dbObj)
 })
+
 setMethod("isIdCurrent", signature(dbObj="MonetDBConnection"), 
def=function(dbObj, ...) {
   .Deprecated("dbIsValid")
    dbIsValid(dbObj)
 })
 
-
 if (is.null(getGeneric("initExtension"))) setGeneric("initExtension", 
function(dbObj, ...) standardGeneric("initExtension"))
 setMethod("initExtension", signature(dbObj="MonetDBConnection"), 
def=function(dbObj, ...) {
   .Deprecated(msg="initExtension() is not required for MonetDBLite")
diff --git a/clients/R/Tests/All b/clients/R/Tests/All
--- a/clients/R/Tests/All
+++ b/clients/R/Tests/All
@@ -6,4 +6,3 @@ HAVE_LIBR?survey
 HAVE_LIBR?dplyr
 HAVE_LIBR?dplyr-flights
 HAVE_LIBR&NOT_WIN32?dbapply
-HAVE_LIBR?testrandom
diff --git a/clients/R/Tests/dbi.R b/clients/R/Tests/dbi.R
--- a/clients/R/Tests/dbi.R
+++ b/clients/R/Tests/dbi.R
@@ -69,7 +69,7 @@ stopifnot(identical(res@env$success,TRUE
 stopifnot(dbColumnInfo(res)[[1,1]] == "Species")
 stopifnot(dbColumnInfo(res)[[2,1]] == "Sepal.Width")
 
-stopifnot(dbGetInfo(res)$row.count == 150 && res@env$info$rows == 150)
+stopifnot(dbGetRowCount(res) == 150 && res@env$info$rows == 150)
 
 data <- dbFetch(res,10)
 
diff --git a/sql/backends/monet5/Tests/simpleSQL.stable.err 
b/sql/backends/monet5/Tests/simpleSQL.stable.err
--- a/sql/backends/monet5/Tests/simpleSQL.stable.err
+++ b/sql/backends/monet5/Tests/simpleSQL.stable.err
@@ -1,11 +1,11 @@
 stderr of test 'simpleSQL` in directory 'sql/backends/monet5` itself:
 
 
-# 23:37:35 >  
-# 23:37:35 >  "mserver5" "--debug=10" "--set" "gdk_nr_threads=0" "--set" 
"mapi_open=true" "--set" "mapi_port=30175" "--set" 
"mapi_usock=/var/tmp/mtest-28801/.s.monetdb.30175" "--set" "monet_prompt=" 
"--forcemito" "--set" "mal_listing=2" 
"--dbpath=/export/scratch1/mk/default//Linux/var/MonetDB/mTests_sql_backends_monet5"
 "simpleSQL.mal"
-# 23:37:35 >  
+# 11:36:51 >  
+# 11:36:51 >  "mserver5" "--debug=10" "--set" "gdk_nr_threads=0" "--set" 
"mapi_open=true" "--set" "mapi_port=37368" "--set" 
"mapi_usock=/var/tmp/mtest-33376/.s.monetdb.37368" "--set" "monet_prompt=" 
"--forcemito" "--dbpath=/tmp/fuckit/var/MonetDB/mTests_sql_backends_monet5" 
"--set" "embedded_r=yes"
+# 11:36:51 >  
 
-# builtin opt  gdk_dbpath = 
/export/scratch1/mk/default//Linux/var/monetdb5/dbfarm/demo
+# builtin opt  gdk_dbpath = /tmp/fuckit/var/monetdb5/dbfarm/demo
 # builtin opt  gdk_debug = 0
 # builtin opt  gdk_vmtrim = no
 # builtin opt  monet_prompt = >
@@ -17,19 +17,19 @@ stderr of test 'simpleSQL` in directory 
 # builtin opt  sql_debug = 0
 # cmdline opt  gdk_nr_threads = 0
 # cmdline opt  mapi_open = true
-# cmdline opt  mapi_port = 30175
-# cmdline opt  mapi_usock = /var/tmp/mtest-28801/.s.monetdb.30175
+# cmdline opt  mapi_port = 37368
+# cmdline opt  mapi_usock = /var/tmp/mtest-33376/.s.monetdb.37368
 # cmdline opt  monet_prompt = 
-# cmdline opt  mal_listing = 2
-# cmdline opt  gdk_dbpath = 
/export/scratch1/mk/default//Linux/var/MonetDB/mTests_sql_backends_monet5
+# cmdline opt  gdk_dbpath = /tmp/fuckit/var/MonetDB/mTests_sql_backends_monet5
+# cmdline opt  embedded_r = yes
 # cmdline opt  gdk_debug = 536870922
 
-# 13:03:24 >  
-# 13:03:24 >  "mclient" "-lmal" "-ftest" "-Eutf-8" 
"--host=/var/tmp/mtest-21282" "--port=38130"
-# 13:03:24 >  
+# 11:36:52 >  
+# 11:36:52 >  "mclient" "-lmal" "-ftest" "-Eutf-8" 
"--host=/var/tmp/mtest-33376" "--port=37368"
+# 11:36:52 >  
 
 
-# 11:30:21 >  
-# 11:30:21 >  "Done."
-# 11:30:21 >  
+# 11:36:52 >  
+# 11:36:52 >  "Done."
+# 11:36:52 >  
 
diff --git a/sql/backends/monet5/sql_execute.c 
b/sql/backends/monet5/sql_execute.c
--- a/sql/backends/monet5/sql_execute.c
+++ b/sql/backends/monet5/sql_execute.c
@@ -207,6 +207,8 @@ SQLstatementIntern(Client c, str *expr, 
 
                if (execute) {
                        MalBlkPtr mb = c->curprg->def;
+                       if (!output)
+                               sql->out = NULL;        /* no output stream */
                        msg = runMAL(c, mb, 0, 0);
                        MSresetInstructions(mb, oldstop);
                        freeVariables(c, mb, NULL, oldvtop);
diff --git a/sql/test/BugTracker-2014/Tests/ifthenelse.Bug-3629.stable.out 
b/sql/test/BugTracker-2014/Tests/ifthenelse.Bug-3629.stable.out
--- a/sql/test/BugTracker-2014/Tests/ifthenelse.Bug-3629.stable.out
+++ b/sql/test/BugTracker-2014/Tests/ifthenelse.Bug-3629.stable.out
@@ -38,7 +38,7 @@ Ready.
 % .L1, .L2,    .L2 # table_name
 % L1,  L2,     test_single_value # name
 % int, int,    int # type
-% 1,   1,      11 # length
+% 1,   1,      1 # length
 [ 1,   0,      NULL    ]
 #select test(true);
 % .L # table_name
diff --git 
a/sql/test/BugTracker-2015/Tests/crash_in_reduce_groupby.Bug-3818.stable.out 
b/sql/test/BugTracker-2015/Tests/crash_in_reduce_groupby.Bug-3818.stable.out
--- a/sql/test/BugTracker-2015/Tests/crash_in_reduce_groupby.Bug-3818.stable.out
+++ b/sql/test/BugTracker-2015/Tests/crash_in_reduce_groupby.Bug-3818.stable.out
@@ -49,7 +49,7 @@ Ready.
 % .L,  .L,     .L,     .L,     .L,     sys.L1 # table_name
 % cods,        elrik,  ether,  jaelen, sora,   L1 # name
 % tinyint,     tinyint,        tinyint,        tinyint,        tinyint,        
bigint # type
-% 1,   1,      1,      1,      1,      20 # length
+% 1,   1,      1,      1,      1,      1 # length
 [ 0,   0,      0,      0,      0,      NULL    ]
 #drop table t2a;
 #drop table t1a;
diff --git 
a/sql/test/BugTracker-2015/Tests/crash_in_reduce_groupby.Bug-3818.stable.out.int128
 
b/sql/test/BugTracker-2015/Tests/crash_in_reduce_groupby.Bug-3818.stable.out.int128
--- 
a/sql/test/BugTracker-2015/Tests/crash_in_reduce_groupby.Bug-3818.stable.out.int128
+++ 
b/sql/test/BugTracker-2015/Tests/crash_in_reduce_groupby.Bug-3818.stable.out.int128
@@ -49,7 +49,7 @@ Ready.
 % .L,  .L,     .L,     .L,     .L,     sys.L1 # table_name
 % cods,        elrik,  ether,  jaelen, sora,   L1 # name
 % tinyint,     tinyint,        tinyint,        tinyint,        tinyint,        
hugeint # type
-% 1,   1,      1,      1,      1,      40 # length
+% 1,   1,      1,      1,      1,      1 # length
 [ 0,   0,      0,      0,      0,      NULL    ]
 #drop table t2a;
 #drop table t1a;
diff --git a/tools/embedded/Tests/dbi.R b/tools/embedded/Tests/dbi.R
--- a/tools/embedded/Tests/dbi.R
+++ b/tools/embedded/Tests/dbi.R
@@ -52,7 +52,7 @@ test_that("import export", {
        expect_true(res@env$success)
        expect_equal(dbColumnInfo(res)[[1,1]], "Species")
        expect_equal(dbColumnInfo(res)[[2,1]], "Sepal.Width")
-       expect_equal(dbGetInfo(res)$row.count, 150)
+       expect_equal(dbGetRowCount(res), 150)
        expect_equal(res@env$info$rows, 150)
 
        data2 <- dbFetch(res,-1)
diff --git a/tools/embedded/build-on-windows.bat 
b/tools/embedded/build-on-windows.bat
--- a/tools/embedded/build-on-windows.bat
+++ b/tools/embedded/build-on-windows.bat
@@ -1,4 +1,5 @@
+rm MonetDBLite_0.2.0.zip
 "c:\Program Files (x86)\GnuWin32\bin\wget.exe" 
http://dev.monetdb.org/Assets/R/MonetDBLite_0.2.0.tar.gz
-"c:\Program Files\R\R-3.2.3\bin\R" CMD INSTALL --build --force-biarch 
MonetDBLite_0.2.0.tar.gz 
+"c:\Program Files\R\R-3.2.3\bin\R" CMD INSTALL --build --merge-multiarch 
MonetDBLite_0.2.0.tar.gz 
 rm MonetDBLite_0.2.0.tar.gz 
 pause
_______________________________________________
checkin-list mailing list
checkin-list@monetdb.org
https://www.monetdb.org/mailman/listinfo/checkin-list

Reply via email to