Changeset: 219c907642b9 for MonetDB
URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=219c907642b9
Modified Files:
        monetdb5/extras/jaql/jaqlgencode.c
Branch: jacqueline
Log Message:

jaql: revert expensive order preserving code

Since we return multiple values as arrays now, the order preserving code
that we had to introduce to work with multiple values for the same
element from d4debd62a329 is no longer necessary.  Revert that part.


diffs (85 lines):

diff --git a/monetdb5/extras/jaql/jaqlgencode.c 
b/monetdb5/extras/jaql/jaqlgencode.c
--- a/monetdb5/extras/jaql/jaqlgencode.c
+++ b/monetdb5/extras/jaql/jaqlgencode.c
@@ -2223,14 +2223,6 @@ dumpvariabletransformation(MalBlkPtr mb,
 
                        /* return in original elem order */
                        q = newInstruction(mb, ASSIGNsymbol);
-                       setModuleId(q, algebraRef);
-                       setFunctionId(q, markHRef);
-                       q = pushReturn(mb, q, newTmpVariable(mb, TYPE_any));
-                       q = pushArgument(mb, q, b);
-                       q = pushOid(mb, q, 0);
-                       g = getArg(q, 0);
-                       pushInstruction(mb, q);
-                       q = newInstruction(mb, ASSIGNsymbol);
                        setModuleId(q, batRef);
                        setFunctionId(q, reverseRef);
                        q = pushReturn(mb, q, newTmpVariable(mb, TYPE_any));
@@ -2239,66 +2231,13 @@ dumpvariabletransformation(MalBlkPtr mb,
                        pushInstruction(mb, q);
                        q = newInstruction(mb, ASSIGNsymbol);
                        setModuleId(q, algebraRef);
-                       setFunctionId(q, markHRef);
-                       q = pushReturn(mb, q, newTmpVariable(mb, TYPE_any));
-                       q = pushArgument(mb, q, b);
-                       q = pushOid(mb, q, 0);
-                       f = getArg(q, 0);
-                       pushInstruction(mb, q);
-                       q = newInstruction(mb, ASSIGNsymbol);
-                       setModuleId(q, batRef);
-                       setFunctionId(q, reverseRef);
-                       q = pushReturn(mb, q, newTmpVariable(mb, TYPE_any));
-                       q = pushArgument(mb, q, g);
-                       g = getArg(q, 0);
-                       pushInstruction(mb, q);
-
-                       q = newInstruction(mb, ASSIGNsymbol);
-                       setModuleId(q, algebraRef);
                        setFunctionId(q, leftjoinRef);
                        q = pushReturn(mb, q, newTmpVariable(mb, TYPE_any));
                        q = pushArgument(mb, q, e);
-                       q = pushArgument(mb, q, g);
-                       b = getArg(q, 0);
-                       pushInstruction(mb, q);
-                       q = newInstruction(mb, ASSIGNsymbol);
-                       setModuleId(q, batRef);
-                       setFunctionId(q, reverseRef);
-                       q = pushReturn(mb, q, newTmpVariable(mb, TYPE_any));
                        q = pushArgument(mb, q, b);
                        b = getArg(q, 0);
                        pushInstruction(mb, q);
                        q = newInstruction(mb, ASSIGNsymbol);
-                       setModuleId(q, algebraRef);
-                       setFunctionId(q, putName("ssort", 5));
-                       q = pushReturn(mb, q, newTmpVariable(mb, TYPE_any));
-                       q = pushArgument(mb, q, b);
-                       b = getArg(q, 0);
-                       pushInstruction(mb, q);
-                       q = newInstruction(mb, ASSIGNsymbol);
-                       setModuleId(q, batRef);
-                       setFunctionId(q, reverseRef);
-                       q = pushReturn(mb, q, newTmpVariable(mb, TYPE_any));
-                       q = pushArgument(mb, q, b);
-                       b = getArg(q, 0);
-                       pushInstruction(mb, q);
-                       q = newInstruction(mb, ASSIGNsymbol);
-                       setModuleId(q, algebraRef);
-                       setFunctionId(q, putName("ssort", 5));
-                       q = pushReturn(mb, q, newTmpVariable(mb, TYPE_any));
-                       q = pushArgument(mb, q, b);
-                       b = getArg(q, 0);
-                       pushInstruction(mb, q);
-                       q = newInstruction(mb, ASSIGNsymbol);
-                       setModuleId(q, algebraRef);
-                       setFunctionId(q, leftjoinRef);
-                       q = pushReturn(mb, q, newTmpVariable(mb, TYPE_any));
-                       q = pushArgument(mb, q, b);
-                       q = pushArgument(mb, q, f);
-                       b = getArg(q, 0);
-                       pushInstruction(mb, q);
-
-                       q = newInstruction(mb, ASSIGNsymbol);
                        setModuleId(q, batRef);
                        setFunctionId(q, reverseRef);
                        q = pushReturn(mb, q, newTmpVariable(mb, TYPE_any));
_______________________________________________
Checkin-list mailing list
Checkin-list@monetdb.org
http://mail.monetdb.org/mailman/listinfo/checkin-list

Reply via email to