Changeset: 9639af5d5c11 for MonetDB
URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=9639af5d5c11
Added Files:
        sql/backends/monet5/Tests/pyapi34.sql
        sql/backends/monet5/Tests/pyapi34.stable.err
        sql/backends/monet5/Tests/pyapi34.stable.out
        sql/test/BugTracker-2017/Tests/crash-dce.Bug-6330.stable.out.int128
        
sql/test/BugTracker-2017/Tests/crash-select_after_MAL_error.Bug-6332.stable.out.int128
        sql/test/BugTracker-2017/Tests/default-insert-update.Bug-6249.stable.err
        sql/test/BugTracker-2017/Tests/default-insert-update.Bug-6249.stable.out
Modified Files:
        gdk/gdk_hash.c
        gdk/gdk_imprints.c
        sql/backends/monet5/Tests/All
        sql/backends/monet5/UDF/pyapi/connection.c
        sql/backends/monet5/UDF/pyapi/conversion.c
        sql/backends/monet5/sql_execute.c
        sql/backends/monet5/sql_gencode.c
        sql/server/rel_dump.c
        sql/server/rel_schema.c
        sql/server/rel_updates.c
        sql/server/sql_parser.y
        sql/storage/store.c
        sql/test/BugTracker-2009/Tests/copy_multiple_files.SF-2902320.stable.out
        
sql/test/BugTracker-2009/Tests/use_order_column_first.SF-2686008.stable.out
        
sql/test/BugTracker-2010/Tests/LIMIT_OFFSET_big-endian.Bug-2622.stable.out
        
sql/test/BugTracker-2010/Tests/ORDER_BY_over_UNION_EXCEPT_INTERSECT.Bug-2606.stable.out
        sql/test/BugTracker-2010/Tests/limit_in_prepare.Bug-2552.sql
        sql/test/BugTracker-2011/Tests/count-count-distinct.Bug-2808.sql
        sql/test/BugTracker-2011/Tests/crash_on_alias.Bug-2798.stable.out
        sql/test/BugTracker-2011/Tests/crash_on_alias.Bug-2798.stable.out.int128
        
sql/test/BugTracker-2012/Tests/aggregate_vs_positional_column_crash.Bug-3085.sql
        sql/test/BugTracker-2012/Tests/predicate_select.Bug-3090.sql
        sql/test/BugTracker-2012/Tests/predicate_select.Bug-3090.stable.out
        sql/test/BugTracker-2013/Tests/qualified_aggrname.Bug-3332.sql
        sql/test/BugTracker-2013/Tests/qualified_aggrname.Bug-3332.stable.out
        sql/test/BugTracker-2013/Tests/rangejoin_optimizer.Bug-3411.stable.out
        sql/test/BugTracker-2013/Tests/swapped_likejoin.Bug-3375.sql
        sql/test/BugTracker-2013/Tests/swapped_likejoin.Bug-3375.stable.out
        sql/test/BugTracker-2014/Tests/select-having.Bug-3458.sql
        sql/test/BugTracker-2015/Tests/cardinality.Bug-3761.sql
        sql/test/BugTracker-2015/Tests/cardinality.Bug-3761.stable.out
        sql/test/BugTracker-2015/Tests/crash.Bug-3736.stable.out
        sql/test/BugTracker-2015/Tests/large_join.Bug-3809.stable.out
        
sql/test/BugTracker-2015/Tests/quantile_function_resolution.Bug-3773.stable.out
        sql/test/BugTracker-2015/Tests/schema_view.Bug-3708.stable.out
        
sql/test/BugTracker-2016/Tests/memory-consumption-query-PLAN-25joins.Bug-3972.stable.out
        sql/test/BugTracker-2016/Tests/merge_project.Bug-3955.stable.out
        sql/test/BugTracker-2017/Tests/All
        sql/test/BugTracker-2017/Tests/crash-dce.Bug-6330.stable.out
        
sql/test/BugTracker-2017/Tests/crash-select_after_MAL_error.Bug-6332.stable.out
        sql/test/BugTracker/Tests/explain.SF-1739353.stable.out
        sql/test/BugTracker/Tests/jdbc_no_debug.SF-1739356.stable.out
        sql/test/Tests/order_by_complex_exp.stable.out
        sql/test/bugs/Tests/crash_order_by.sql
        sql/test/bugs/Tests/simple_view.sql
        sql/test/bugs/insert_delete-bug-sf-904025.sql
        sql/test/leaks/Tests/check1.stable.out.int128
        sql/test/mergetables/Tests/part-elim.stable.out
        sql/test/pg_regress/Tests/alter_table.stable.err
        sql/test/pg_regress/Tests/alter_table.stable.out
        sql/test/pg_regress/Tests/insert.stable.err
        tools/merovingian/client/monetdb.c
        tools/merovingian/daemon/merovingian.c
        tools/merovingian/utils/properties.c
Branch: stratified_sampling
Log Message:

Merge with default.


diffs (truncated from 3526 to 300 lines):

diff --git a/gdk/gdk_hash.c b/gdk/gdk_hash.c
--- a/gdk/gdk_hash.c
+++ b/gdk/gdk_hash.c
@@ -202,17 +202,16 @@ BATcheckhash(BAT *b)
                Hash *h;
                Heap *hp;
                const char *nme = BBP_physical(b->batCacheid);
-               const char *ext = b->batCacheid > 0 ? "thash" : "hhash";
                int fd;
 
                b->thash = NULL;
                if ((hp = GDKzalloc(sizeof(*hp))) != NULL &&
                    (hp->farmid = BBPselectfarm(b->batRole, b->ttype, 
hashheap)) >= 0 &&
                    (hp->filename = GDKmalloc(strlen(nme) + 12)) != NULL) {
-                       sprintf(hp->filename, "%s.%s", nme, ext);
+                       sprintf(hp->filename, "%s.thash", nme);
 
                        /* check whether a persisted hash can be found */
-                       if ((fd = GDKfdlocate(hp->farmid, nme, "rb+", ext)) >= 
0) {
+                       if ((fd = GDKfdlocate(hp->farmid, nme, "rb+", "thash")) 
>= 0) {
                                size_t hdata[HASH_HEADER_SIZE];
                                struct stat st;
 
@@ -226,7 +225,7 @@ BATcheckhash(BAT *b)
                                    hdata[4] == (size_t) BATcount(b) &&
                                    fstat(fd, &st) == 0 &&
                                    st.st_size >= (off_t) (hp->size = hp->free 
= (hdata[1] + hdata[2]) * hdata[3] + HASH_HEADER_SIZE * SIZEOF_SIZE_T) &&
-                                   HEAPload(hp, nme, ext, 0) == GDK_SUCCEED) {
+                                   HEAPload(hp, nme, "thash", 0) == 
GDK_SUCCEED) {
                                        h->lim = (BUN) hdata[1];
                                        h->type = ATOMtype(b->ttype);
                                        h->mask = (BUN) (hdata[2] - 1);
@@ -260,7 +259,7 @@ BATcheckhash(BAT *b)
                                GDKfree(h);
                                close(fd);
                                /* unlink unusable file */
-                               GDKunlink(hp->farmid, BATDIR, nme, ext);
+                               GDKunlink(hp->farmid, BATDIR, nme, "thash");
                        }
                        GDKfree(hp->filename);
                }
@@ -338,7 +337,6 @@ BAThash(BAT *b, BUN masksize)
                Hash *h = NULL;
                Heap *hp;
                const char *nme = BBP_physical(b->batCacheid);
-               const char *ext = b->batCacheid > 0 ? "thash" : "hhash";
                BATiter bi = bat_iterator(b);
 
                ALGODEBUG fprintf(stderr, "#BAThash: create hash(%s#" BUNFMT 
");\n", BATgetId(b), BATcount(b));
@@ -350,7 +348,7 @@ BAThash(BAT *b, BUN masksize)
                        return GDK_FAIL;
                }
                hp->dirty = TRUE;
-               sprintf(hp->filename, "%s.%s", nme, ext);
+               sprintf(hp->filename, "%s.thash", nme);
 
                /* cnt = 0, hopefully there is a proper capacity from
                 * which we can derive enough information */
diff --git a/gdk/gdk_imprints.c b/gdk/gdk_imprints.c
--- a/gdk/gdk_imprints.c
+++ b/gdk/gdk_imprints.c
@@ -208,7 +208,6 @@ BATcheckimprints(BAT *b)
                Imprints *imprints;
                Heap *hp;
                str nme = BBP_physical(b->batCacheid);
-               const char *ext = b->batCacheid > 0 ? "timprints" : "himprints";
 
                b->timprints = NULL;
                if ((hp = GDKzalloc(sizeof(Heap))) != NULL &&
@@ -216,10 +215,10 @@ BATcheckimprints(BAT *b)
                    (hp->filename = GDKmalloc(strlen(nme) + 12)) != NULL) {
                        int fd;
 
-                       sprintf(hp->filename, "%s.%s", nme, ext);
+                       sprintf(hp->filename, "%s.timprints", nme);
                        /* check whether a persisted imprints index
                         * can be found */
-                       if ((fd = GDKfdlocate(hp->farmid, nme, "rb", ext)) >= 
0) {
+                       if ((fd = GDKfdlocate(hp->farmid, nme, "rb", 
"timprints")) >= 0) {
                                size_t hdata[4];
                                struct stat st;
                                size_t pages;
@@ -240,7 +239,7 @@ BATcheckimprints(BAT *b)
                                                           hdata[2] * 
sizeof(cchdc_t) +
                                                           sizeof(uint64_t) /* 
padding for alignment */
                                                           + 4 * SIZEOF_SIZE_T) 
&&
-                                   HEAPload(hp, nme, b->batCacheid > 0 ? 
"timprints" : "himprints", 0) == GDK_SUCCEED) {
+                                   HEAPload(hp, nme, "timprints", 0) == 
GDK_SUCCEED) {
                                        /* usable */
                                        imprints->imprints = hp;
                                        imprints->bits = (bte) (hdata[0] & 
0xFF);
@@ -261,7 +260,7 @@ BATcheckimprints(BAT *b)
                                GDKfree(imprints);
                                close(fd);
                                /* unlink unusable file */
-                               GDKunlink(hp->farmid, BATDIR, nme, ext);
+                               GDKunlink(hp->farmid, BATDIR, nme, "timprints");
                        }
                        GDKfree(hp->filename);
                }
@@ -342,8 +341,7 @@ BATimprints(BAT *b)
                        MT_lock_unset(&GDKimprintsLock(b->batCacheid));
                        return GDK_FAIL;
                }
-               sprintf(imprints->imprints->filename, "%s.%cimprints", nme,
-                       b->batCacheid > 0 ? 't' : 'h');
+               sprintf(imprints->imprints->filename, "%s.timprints", nme);
                pages = (((size_t) BATcount(b) * b->twidth) + IMPS_PAGE - 1) / 
IMPS_PAGE;
                imprints->imprints->farmid = BBPselectfarm(b->batRole, b->ttype,
                                                           imprintsheap);
@@ -472,9 +470,9 @@ BATimprints(BAT *b)
                ((size_t *) imprints->imprints->base)[2] = (size_t) 
imprints->dictcnt;
                ((size_t *) imprints->imprints->base)[3] = (size_t) BATcount(b);
                if ((BBP_status(b->batCacheid) & BBPEXISTING) &&
-                   HEAPsave(imprints->imprints, nme, b->batCacheid > 0 ? 
"timprints" : "himprints") == GDK_SUCCEED &&
+                   HEAPsave(imprints->imprints, nme, "timprints") == 
GDK_SUCCEED &&
                    (fd = GDKfdlocate(imprints->imprints->farmid, nme, "rb+",
-                                     b->batCacheid > 0 ? "timprints" : 
"himprints")) >= 0) {
+                                     "timprints")) >= 0) {
                        ALGODEBUG fprintf(stderr, "#BATimprints: persisting 
imprints\n");
                        /* add version number */
                        ((size_t *) imprints->imprints->base)[0] |= (size_t) 
IMPRINTS_VERSION << 8;
@@ -610,7 +608,7 @@ IMPSremove(BAT *b)
                    * (size_t *) imprints->imprints->base & (1 << 16))
                        fprintf(stderr, "#IMPSremove: removing persisted 
imprints\n");
                if (HEAPdelete(imprints->imprints, BBP_physical(b->batCacheid),
-                              b->batCacheid > 0 ? "timprints" : "himprints"))
+                              "timprints"))
                        IODEBUG fprintf(stderr, "#IMPSremove(%s): imprints 
heap\n", BATgetId(b));
 
                GDKfree(imprints->imprints);
diff --git a/sql/backends/monet5/Tests/All b/sql/backends/monet5/Tests/All
--- a/sql/backends/monet5/Tests/All
+++ b/sql/backends/monet5/Tests/All
@@ -61,6 +61,7 @@ HAVE_LIBPY?pyapi29
 HAVE_LIBPY?pyapi30
 HAVE_LIBPY?pyapi32
 HAVE_LIBPY?pyapi33
+HAVE_LIBPY?pyapi34
 
 HAVE_LIBPY?pyloader01
 HAVE_LIBPY?pyloader02
diff --git a/sql/backends/monet5/Tests/pyapi34.sql 
b/sql/backends/monet5/Tests/pyapi34.sql
new file mode 100644
--- /dev/null
+++ b/sql/backends/monet5/Tests/pyapi34.sql
@@ -0,0 +1,18 @@
+
+START TRANSACTION;
+
+CREATE TABLE booleans(a BOOLEAN);
+INSERT INTO booleans VALUES (1), (0), (1);
+
+CREATE FUNCTION pyapi34a(inp BOOLEAN) RETURNS BOOLEAN LANGUAGE PYTHON {
+       results = _conn.execute(u'SELECT * FROM booleans;')
+       return {'result': numpy.logical_xor(inp, results['a']) };
+};
+
+CREATE FUNCTION pyapi34b(inp BOOLEAN) RETURNS BOOLEAN LANGUAGE PYTHON {
+       return {'result': inp};
+};
+
+SELECT a, pyapi34a(a), pyapi34b(a) FROM booleans;
+
+ROLLBACK;
diff --git a/sql/backends/monet5/Tests/pyapi34.stable.err 
b/sql/backends/monet5/Tests/pyapi34.stable.err
new file mode 100644
--- /dev/null
+++ b/sql/backends/monet5/Tests/pyapi34.stable.err
@@ -0,0 +1,35 @@
+stderr of test 'pyapi34` in directory 'sql/backends/monet5` itself:
+
+
+# 11:20:10 >  
+# 11:20:10 >  "mserver5" "--debug=10" "--set" "gdk_nr_threads=0" "--set" 
"mapi_open=true" "--set" "mapi_port=37278" "--set" 
"mapi_usock=/var/tmp/mtest-61818/.s.monetdb.37278" "--set" "monet_prompt=" 
"--forcemito" "--dbpath=/Users/myth/opt/var/MonetDB/mTests_sql_backends_monet5" 
"--set" "embedded_py=true"
+# 11:20:10 >  
+
+# builtin opt  gdk_dbpath = /Users/myth/opt/var/monetdb5/dbfarm/demo
+# builtin opt  gdk_debug = 0
+# builtin opt  gdk_vmtrim = no
+# builtin opt  monet_prompt = >
+# builtin opt  monet_daemon = no
+# builtin opt  mapi_port = 50000
+# builtin opt  mapi_open = false
+# builtin opt  mapi_autosense = false
+# builtin opt  sql_optimizer = default_pipe
+# builtin opt  sql_debug = 0
+# cmdline opt  gdk_nr_threads = 0
+# cmdline opt  mapi_open = true
+# cmdline opt  mapi_port = 37278
+# cmdline opt  mapi_usock = /var/tmp/mtest-61818/.s.monetdb.37278
+# cmdline opt  monet_prompt = 
+# cmdline opt  gdk_dbpath = 
/Users/myth/opt/var/MonetDB/mTests_sql_backends_monet5
+# cmdline opt  embedded_py = true
+# cmdline opt  gdk_debug = 536870922
+
+# 11:20:10 >  
+# 11:20:10 >  "mclient" "-lsql" "-ftest" "-Eutf-8" "-i" "-e" 
"--host=/var/tmp/mtest-61818" "--port=37278"
+# 11:20:10 >  
+
+
+# 11:20:10 >  
+# 11:20:10 >  "Done."
+# 11:20:10 >  
+
diff --git a/sql/backends/monet5/Tests/pyapi34.stable.out 
b/sql/backends/monet5/Tests/pyapi34.stable.out
new file mode 100644
--- /dev/null
+++ b/sql/backends/monet5/Tests/pyapi34.stable.out
@@ -0,0 +1,86 @@
+stdout of test 'pyapi34` in directory 'sql/backends/monet5` itself:
+
+
+# 11:20:10 >  
+# 11:20:10 >  "mserver5" "--debug=10" "--set" "gdk_nr_threads=0" "--set" 
"mapi_open=true" "--set" "mapi_port=37278" "--set" 
"mapi_usock=/var/tmp/mtest-61818/.s.monetdb.37278" "--set" "monet_prompt=" 
"--forcemito" "--dbpath=/Users/myth/opt/var/MonetDB/mTests_sql_backends_monet5" 
"--set" "embedded_py=true"
+# 11:20:10 >  
+
+# MonetDB 5 server v11.27.0
+# This is an unreleased version
+# Serving database 'mTests_sql_backends_monet5', using 4 threads
+# Compiled for x86_64-apple-darwin15.6.0/64bit with 128bit integers
+# Found 8.000 GiB available main-memory.
+# Copyright (c) 1993-July 2008 CWI.
+# Copyright (c) August 2008-2017 MonetDB B.V., all rights reserved
+# Visit https://www.monetdb.org/ for further information
+# Listening for connection requests on 
mapi:monetdb://dhcp-21.eduroam.cwi.nl:37278/
+# Listening for UNIX domain connection requests on 
mapi:monetdb:///var/tmp/mtest-61818/.s.monetdb.37278
+# MonetDB/SQL module loaded
+# MonetDB/Python2 module loaded
+
+Ready.
+# 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_temporal.sql
+# loading sql script: 18_index.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: 25_debug.sql
+# loading sql script: 26_sysmon.sql
+# loading sql script: 27_rejects.sql
+# loading sql script: 39_analytics.sql
+# loading sql script: 39_analytics_hge.sql
+# loading sql script: 40_json.sql
+# loading sql script: 40_json_hge.sql
+# loading sql script: 41_md5sum.sql
+# loading sql script: 45_uuid.sql
+# loading sql script: 46_profiler.sql
+# loading sql script: 51_sys_schema_extension.sql
+# loading sql script: 72_fits.sql
+# loading sql script: 74_netcdf.sql
+# loading sql script: 75_storagemodel.sql
+# loading sql script: 80_statistics.sql
+# loading sql script: 80_udf.sql
+# loading sql script: 80_udf_hge.sql
+# loading sql script: 90_generator.sql
+# loading sql script: 90_generator_hge.sql
+# loading sql script: 99_system.sql
+
+# 11:20:10 >  
+# 11:20:10 >  "mclient" "-lsql" "-ftest" "-Eutf-8" "-i" "-e" 
"--host=/var/tmp/mtest-61818" "--port=37278"
+# 11:20:10 >  
+
+#START TRANSACTION;
+#CREATE TABLE booleans(a BOOLEAN);
+#INSERT INTO booleans VALUES (1), (0), (1);
+[ 3    ]
+#CREATE FUNCTION pyapi34a(inp BOOLEAN) RETURNS BOOLEAN LANGUAGE PYTHON {
+#      results = _conn.execute('SELECT * FROM booleans;')
+#      return {'result': numpy.logical_xor(inp, results['a']) };
+#};
+#CREATE FUNCTION pyapi34b(inp BOOLEAN) RETURNS BOOLEAN LANGUAGE PYTHON {
+#      return {'result': inp};
+#};
+#SELECT a, pyapi34a(a), pyapi34b(a) FROM booleans;
+% sys.booleans,        sys.L3, sys.L5 # table_name
+% a,   L3,     L5 # name
+% boolean,     boolean,        boolean # type
+% 5,   5,      5 # length
+[ true,        false,  true    ]
+[ false,       false,  false   ]
+[ true,        false,  true    ]
+#ROLLBACK;
+
+# 11:20:10 >  
+# 11:20:10 >  "Done."
+# 11:20:10 >  
+
diff --git a/sql/backends/monet5/UDF/pyapi/connection.c 
b/sql/backends/monet5/UDF/pyapi/connection.c
--- a/sql/backends/monet5/UDF/pyapi/connection.c
+++ b/sql/backends/monet5/UDF/pyapi/connection.c
@@ -19,26 +19,44 @@ CREATE_SQL_FUNCTION_PTR(str, create_tabl
_______________________________________________
checkin-list mailing list
[email protected]
https://www.monetdb.org/mailman/listinfo/checkin-list

Reply via email to