Changeset: c7f0e92444a9 for MonetDB
URL: https://dev.monetdb.org/hg/MonetDB/rev/c7f0e92444a9
Modified Files:
        monetdb5/optimizer/opt_dataflow.c
Branch: parappend
Log Message:

Reorder clauses to improve hit ratio

Column names are more likely to differ than schema- or table names.


diffs (29 lines):

diff --git a/monetdb5/optimizer/opt_dataflow.c 
b/monetdb5/optimizer/opt_dataflow.c
--- a/monetdb5/optimizer/opt_dataflow.c
+++ b/monetdb5/optimizer/opt_dataflow.c
@@ -253,9 +253,9 @@ sqlBreakpoint(MalBlkPtr mb, InstrPtr *fi
        str my_tname = get_sql_tname(mb, instr);
        str my_cname = get_sql_cname(mb, instr);
        for (InstrPtr *q = first; q < p; q++) {
-               str sname = get_sql_sname(mb, *q);
-               if (strcmp(my_sname, sname) != 0) {
-                       // different sname, no conflict
+               str cname = get_sql_cname(mb, *q);
+               if (strcmp(my_cname, cname) != 0) {
+                       // different cname, no conflict
                        continue;
                }
                str tname = get_sql_tname(mb, *q);
@@ -263,9 +263,9 @@ sqlBreakpoint(MalBlkPtr mb, InstrPtr *fi
                        // different tname, no conflict
                        continue;
                }
-               str cname = get_sql_cname(mb, *q);
-               if (strcmp(my_cname, cname) != 0) {
-                       // different cname, no conflict
+               str sname = get_sql_sname(mb, *q);
+               if (strcmp(my_sname, sname) != 0) {
+                       // different sname, no conflict
                        continue;
                }
                // Found a statement in the region that works on the same 
column so this is a breakpoint
_______________________________________________
checkin-list mailing list
[email protected]
https://www.monetdb.org/mailman/listinfo/checkin-list

Reply via email to