Changeset: 65975e1f56f2 for MonetDB
URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=65975e1f56f2
Removed Files:
        monetdb5/tests/BugReports/Tests/no.119.mal
        monetdb5/tests/BugReports/Tests/no.119.stable.err
        monetdb5/tests/BugReports/Tests/no.119.stable.out
        monetdb5/tests/BugTracker/Tests/thetajoin-properties.Bug-2974.mal
        monetdb5/tests/BugTracker/Tests/thetajoin-properties.Bug-2974.stable.err
        monetdb5/tests/BugTracker/Tests/thetajoin-properties.Bug-2974.stable.out
Modified Files:
        monetdb5/modules/kernel/algebra.mal
        monetdb5/tests/BugReports/Tests/All
        monetdb5/tests/BugTracker/Tests/All
Branch: default
Log Message:

Remove old outerjoin, thetajoin, bandjoin


diffs (truncated from 374 to 300 lines):

diff --git a/monetdb5/modules/kernel/algebra.mal 
b/monetdb5/modules/kernel/algebra.mal
--- a/monetdb5/modules/kernel/algebra.mal
+++ b/monetdb5/modules/kernel/algebra.mal
@@ -83,42 +83,7 @@ command leftfetchjoin ( left:bat[:oid,:o
 address ALGleftfetchjoin
 comment "Hook directly into the left fetch join implementation.";
 
-#command outerjoin( outer:bat[:any_1,:any_2], inner:bat[:any_2,:any_3]) 
:bat[:any_1,:any_3]
-#address ALGouterjoin
-#comment "Returns all the result of a join, plus the BUNS formed NIL in
-#              the tail and the head-values of 'outer' whose tail-value does
-#              not match an head-value in 'inner'.";
-#
-#command outerjoin( outer:bat[:any_1,:any_2], inner:bat[:any_2,:any_3], 
estimate:lng) :bat[:any_1,:any_3]
-#address ALGouterjoinestimate;
-
 # Theta Join
-command thetajoin( left:bat[:any_1,:any_2], right:bat[:any_2,:any_3], 
opname:int) :bat[:any_1,:any_3]
-address ALGthetajoin
-comment "Theta join on for 'mode' in { LE, LT, EQ, GT, GE }.  JOIN_EQ is
-               just the same as join(). All other options do merge algorithms.
-               Either using the fact that they are ordered() already (left on 
tail,
-       right on head), or by using/creating binary search trees on the
-               join columns. ";
-
-command thetajoin( left:bat[:any_1,:any_2], right:bat[:any_2,:any_3], 
opname:int,estimate:lng) :bat[:any_1,:any_3]
-address ALGthetajoinEstimate;
-# Band Join (approximate match)
-command bandjoin( outer:bat[:any_1,:any_2], inner:bat[:any_2,:any_3], 
minus:any_2 , plus:any_2 ) :bat[:any_1,:any_3]
-address ALGbandjoin_default
-comment "This is a join() for which the predicate is that two BUNs match
-               if the left-tail value is within the range [right-head - minus,
-               right-head + plus]. Works only for the builtin numerical types,
-               and their derivates.";
-
-command bandjoin( outer:bat[:any_1,:any_2], inner:bat[:any_2,:any_3], 
minus:any_2 , plus:any_2, li:bit, hi:bit ) :bat[:any_1,:any_3]
-address ALGbandjoin
-comment "This is a join() for which the predicate is that two BUNs match
-               if the left-tail value is within the range [right-head - minus,
-               right-head + plus], depending on (l_in/h_in), the bounds
-               are included. Works only for the builtin numerical types,
-               and their derivates.";
-
 command join(l:bat[:any_1,:any_2], rl:bat[:any_3,:any_2], 
rh:bat[:any_3,:any_2], li:bit, hi:bit) :bat[:any_1,:any_3]
 address ALGrangejoin;
 
@@ -320,8 +285,7 @@ address ALGtunion;
 # @+ Join operations
 # The core of every relational engine.
 # The join collection provided by the GDK kernel.
-command antijoin( left:bat[:oid,:any_1], right:bat[:oid,:any_1])
-               (l:bat[:oid,:oid],r:bat[:oid,:oid])
+command antijoin( left:bat[:oid,:any_1], right:bat[:oid,:any_1]) 
(l:bat[:oid,:oid],r:bat[:oid,:oid])
 address ALGantijoin2
 comment "Returns 2 columns with all BUNs, consisting of the head-oids
          from 'left' and 'right' for which there are BUNs in 'left'
@@ -335,12 +299,12 @@ comment "Returns 2 columns with all BUNs
          and 'right' with equal tails";
 
 # Theta Join
-command thetajoin( left:bat[:oid,:any_1], right:bat[:oid,:any_1], opname:int)
-               (l:bat[:oid,:oid],r:bat[:oid,:oid])
-address ALGthetajoin2
-comment "Returns 2 columns with all BUNs, consisting of the head-oids
-         from 'left' and 'right' for which there are BUNs in 'left'
-         and 'right' with equal tails";
+#command thetajoin( left:bat[:oid,:any_1], right:bat[:oid,:any_1], opname:int)
+#              (l:bat[:oid,:oid],r:bat[:oid,:oid])
+#address ALGthetajoin2
+#comment "Returns 2 columns with all BUNs, consisting of the head-oids
+#        from 'left' and 'right' for which there are BUNs in 'left'
+#        and 'right' with equal tails";
 
 command crossproduct( left:bat[:oid,:any_1], right:bat[:oid,:any_2])
                (l:bat[:oid,:oid],r:bat[:oid,:oid])
@@ -349,15 +313,6 @@ comment "Returns 2 columns with all BUNs
          from 'left' and 'right' for which there are BUNs in 'left'
          and 'right' with equal tails";
 
-command bandjoin( outer:bat[:oid,:any_2], inner:bat[:oid,:any_2], minus:any_2 
, plus:any_2, li:bit, hi:bit )
-               (l:bat[:oid,:oid],r:bat[:oid,:oid])
-address ALGbandjoin2
-comment "This is a join() for which the predicate is that two BUNs match
-               if the left-tail value is within the range [right-head - minus,
-               right-head + plus], depending on (l_in/h_in), the bounds
-               are included. Works only for the builtin numerical types,
-               and their derivates.";
-
 command join(left:bat[:oid,:any_2], rl:bat[:oid,:any_2], rh:bat[:oid,:any_2], 
li:bit, hi:bit)
                (l:bat[:oid,:oid],r:bat[:oid,:oid])
 address ALGrangejoin2;
diff --git a/monetdb5/tests/BugReports/Tests/All 
b/monetdb5/tests/BugReports/Tests/All
--- a/monetdb5/tests/BugReports/Tests/All
+++ b/monetdb5/tests/BugReports/Tests/All
@@ -12,7 +12,6 @@ no.051
 no.062
 no.098
 no.100
-no.119
 no.150
 no.157
 no.161
diff --git a/monetdb5/tests/BugReports/Tests/no.119.mal 
b/monetdb5/tests/BugReports/Tests/no.119.mal
deleted file mode 100644
--- a/monetdb5/tests/BugReports/Tests/no.119.mal
+++ /dev/null
@@ -1,26 +0,0 @@
-       a := bat.new(nil:oid,nil:int);
-       ca_a := bat.append(a,3);
-       da_a := bat.append(a,2);
-       ea_a := bat.append(a,1);
-       ra:= bat.reverse(a);
-       LT:= -1;
-       fa_a := calc.lng(nil);
-       ga_a := algebra.thetajoin(a,ra,LT,fa_a);
-       ha_a := io.print(ga_a);
-       LE:= -2;
-       ia_a := calc.lng(nil);
-       ja_a := algebra.thetajoin(a,ra,LE,ia_a);
-       ka_a := io.print(ja_a);
-       EQ:= 0;
-       la_a := calc.lng(nil);
-       ma_a := algebra.thetajoin(a,ra,EQ,la_a);
-       na_a := io.print(ma_a);
-       GE:= 2;
-       oa_a := calc.lng(nil);
-       pa_a := algebra.thetajoin(a,ra,GE,oa_a);
-       qa_a := io.print(pa_a);
-       GT:= 1;
-       ra_a := calc.lng(nil);
-       sa_a := algebra.thetajoin(a,ra,GT,ra_a);
-       ta_a := io.print(sa_a);
-       ua_a := clients.quit();
diff --git a/monetdb5/tests/BugReports/Tests/no.119.stable.err 
b/monetdb5/tests/BugReports/Tests/no.119.stable.err
deleted file mode 100644
--- a/monetdb5/tests/BugReports/Tests/no.119.stable.err
+++ /dev/null
@@ -1,12 +0,0 @@
-stderr of test 'no.119` in directory 'monetdb5/tests/BugReports` itself:
-
-
-# 07:27:15 >  
-# 07:27:15 >  Mtimeout -timeout 60 mserver5 
"--config=/ufs/mk/monet5/Linux/etc/MonetDB5.conf" --debug=10 --set 
"monet_mod_path=/ufs/mk/monet5/Linux/lib/MonetDB5:/ufs/mk/opensource/MonetDB/Linux/lib/MonetDB"
 --set "gdk_dbfarm=/ufs/mk/monet5/Linux/var/MonetDB5/dbfarm" --set 
"sql_logdir=/ufs/mk/monet5/Linux/var/MonetDB5/dblogs" --set mapi_port=35545 
--set monet_prompt= --trace --dbname=mTests_tests_BugReports  no.119.mal < 
/dev/null
-# 07:27:15 >  
-
-
-# 07:27:15 >  
-# 07:27:15 >  Done.
-# 07:27:15 >  
-
diff --git a/monetdb5/tests/BugReports/Tests/no.119.stable.out 
b/monetdb5/tests/BugReports/Tests/no.119.stable.out
deleted file mode 100644
--- a/monetdb5/tests/BugReports/Tests/no.119.stable.out
+++ /dev/null
@@ -1,94 +0,0 @@
-stdout of test 'no.119` in directory 'monetdb5/tests/BugReports` itself:
-
-
-# 07:27:15 >  
-# 07:27:15 >  Mtimeout -timeout 60 mserver5 
"--config=/ufs/mk/monet5/Linux/etc/MonetDB5.conf" --debug=10 --set 
"monet_mod_path=/ufs/mk/monet5/Linux/lib/MonetDB5:/ufs/mk/opensource/MonetDB/Linux/lib/MonetDB"
 --set "gdk_dbfarm=/ufs/mk/monet5/Linux/var/MonetDB5/dbfarm" --set 
"sql_logdir=/ufs/mk/monet5/Linux/var/MonetDB5/dblogs" --set mapi_port=35545 
--set monet_prompt= --trace --dbname=mTests_tests_BugReports  no.119.mal < 
/dev/null
-# 07:27:15 >  
-
-# MonetDB Server v4.99.19
-# Copyright (c) 1993-2006 CWI, all rights reserved
-# Compiled for x86_64-redhat-linux-gnu/64bit with 64bit OIDs dynamically linked
-# config:/ufs/mk/monet5/Linux/etc/MonetDB5.conf
-# dbfarm:/ufs/mk/monet5/Linux/var/MonetDB5/dbfarm
-# dbname:mTests_tests_BugReports
-# Visit http://monetdb.cwi.nl/ for further information
-# Listening for connection requests on mapi:monetdb://eir.ins.cwi.nl:30579/
-function user.main():void;
-    a := bat.new(nil:oid,nil:int);
-    ca_a := bat.append(a,3);
-    da_a := bat.append(a,2);
-    ea_a := bat.append(a,1);
-    ra := bat.reverse(a);
-    LT := -1;
-    fa_a := calc.lng(nil);
-    ga_a := algebra.thetajoin(a,ra,LT,fa_a);
-    ha_a := io.print(ga_a);
-#      io.print("a.thetajoin(a,LE,calc.lng(nil)).print();"); 
-    LE := -2;
-    ia_a := calc.lng(nil);
-    ja_a := algebra.thetajoin(a,ra,LE,ia_a);
-    ka_a := io.print(ja_a);
-#      io.print("a.thetajoin(a,EQ,calc.lng(nil)).print();"); 
-    EQ := 0;
-    la_a := calc.lng(nil);
-    ma_a := algebra.thetajoin(a,ra,EQ,la_a);
-    na_a := io.print(ma_a);
-#      io.print("a.thetajoin(a,GE,calc.lng(nil)).print();"); 
-    GE := 2;
-    oa_a := calc.lng(nil);
-    pa_a := algebra.thetajoin(a,ra,GE,oa_a);
-    qa_a := io.print(pa_a);
-#      io.print("a.thetajoin(a,GT,calc.lng(nil)).print();"); 
-    GT := 1;
-    ra_a := calc.lng(nil);
-    sa_a := algebra.thetajoin(a,ra,GT,ra_a);
-    ta_a := io.print(sa_a);
-#      io.print("quit();"); 
-    ua_a := clients.quit();
-end main;
-#-----------------#
-# h    t         # name
-# int  int       # type
-#-----------------#
-[ 1@0,   0@0     ]
-[ 2@0,   0@0     ]
-[ 2@0,   1@0     ]
-#-----------------#
-# h    t         # name
-# int  int       # type
-#-----------------#
-[ 0@0,   0@0     ]
-[ 1@0,   0@0     ]
-[ 1@0,   1@0     ]
-[ 2@0,   0@0     ]
-[ 2@0,   1@0     ]
-[ 2@0,   2@0     ]
-#-----------------#
-# h    t         # name
-# int  int       # type
-#-----------------#
-[ 0@0,   0@0     ]
-[ 1@0,   1@0     ]
-[ 2@0,   2@0     ]
-#-----------------#
-# h    t         # name
-# int  int       # type
-#-----------------#
-[ 0@0,   0@0     ]
-[ 0@0,   1@0     ]
-[ 0@0,   2@0     ]
-[ 1@0,   1@0     ]
-[ 1@0,   2@0     ]
-[ 2@0,   2@0     ]
-#-----------------#
-# h    t         # name
-# int  int       # type
-#-----------------#
-[ 0@0,   1@0     ]
-[ 0@0,   2@0     ]
-[ 1@0,   2@0     ]
-
-# 07:27:15 >  
-# 07:27:15 >  Done.
-# 07:27:15 >  
-
diff --git a/monetdb5/tests/BugTracker/Tests/All 
b/monetdb5/tests/BugTracker/Tests/All
--- a/monetdb5/tests/BugTracker/Tests/All
+++ b/monetdb5/tests/BugTracker/Tests/All
@@ -6,6 +6,5 @@ empty_line_crash.SF-1806638
 parser.SF-2051309
 mal_var_persistent_over_sessions.SF-2581675
 joinpath-loop.Bug-2965
-thetajoin-properties.Bug-2974
 algebra_project.Bug-3045
 stdev.Bug-3178
diff --git a/monetdb5/tests/BugTracker/Tests/thetajoin-properties.Bug-2974.mal 
b/monetdb5/tests/BugTracker/Tests/thetajoin-properties.Bug-2974.mal
deleted file mode 100644
--- a/monetdb5/tests/BugTracker/Tests/thetajoin-properties.Bug-2974.mal
+++ /dev/null
@@ -1,19 +0,0 @@
-l := bat.new(:oid, :int);
-r := bat.new(:oid, :int);
-
-l := bat.insert(l, 2@0, 1);
-l := bat.insert(l, 6@0, 2);
-
-r := bat.insert(r, 3@0, 1);
-r := bat.insert(r, 7@0, 2);
-
-rr := bat.reverse(r);
-
-t := algebra.thetajoin(l, rr, 1);  # gt
-
-io.print(t);  # returns 6@0 3@0  (2 is greater than 1?)
-
-b := bat.new(:oid, :oid);
-b := bat.insert(b, t);  # BUG: ERROR: BATkey: dense column must be unique.
-
-io.print(b);
diff --git 
a/monetdb5/tests/BugTracker/Tests/thetajoin-properties.Bug-2974.stable.err 
b/monetdb5/tests/BugTracker/Tests/thetajoin-properties.Bug-2974.stable.err
deleted file mode 100644
--- a/monetdb5/tests/BugTracker/Tests/thetajoin-properties.Bug-2974.stable.err
+++ /dev/null
@@ -1,31 +0,0 @@
-stderr of test 'thetajoin-properties.Bug-2974` in directory 
'monetdb5/tests/BugTracker` itself:
-
-
-# 16:24:34 >  
-# 16:24:34 >  "mserver5" "--debug=10" "--set" "gdk_nr_threads=0" "--set" 
"gdk_dbfarm=/ufs/sjoerd/Monet-stable/var/MonetDB" "--set" "mapi_open=true" 
"--set" "mapi_port=37407" "--set" "monet_prompt=" "--trace" "--forcemito" 
"--set" "mal_listing=2" "--dbname=mTests_tests_BugTracker" 
"thetajoin-properties.Bug-2974.mal"
-# 16:24:34 >  
-
-# builtin opt  gdk_dbname = demo
-# builtin opt  gdk_dbfarm = /ufs/sjoerd/Monet-stable/var/monetdb5/dbfarm
-# builtin opt  gdk_debug = 0
-# builtin opt  gdk_alloc_map = no
-# builtin opt  gdk_vmtrim = yes
_______________________________________________
checkin-list mailing list
checkin-list@monetdb.org
https://www.monetdb.org/mailman/listinfo/checkin-list

Reply via email to