MonetDB: mal-weld - weld groupby: we can use just the str offset...

[Mihai Varga]

Changeset: 8c04ae667582 for MonetDB
URL: https://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=8c04ae667582
Modified Files:
        monetdb5/modules/mal/mal_weld.c
Branch: mal-weld
Log Message:

weld groupby: we can use just the str offset as key for a string col


diffs (23 lines):

diff --git a/monetdb5/modules/mal/mal_weld.c b/monetdb5/modules/mal/mal_weld.c
--- a/monetdb5/modules/mal/mal_weld.c
+++ b/monetdb5/modules/mal/mal_weld.c
@@ -1045,16 +1045,16 @@ WeldGroup(Client cntxt, MalBlkPtr mb, Ma
                        dep = NULL;
                }
                sprintf(zipStmt + strlen(zipStmt), "v%d,", col);
-               sprintf(dictTypeStmt + strlen(dictTypeStmt), " %s,", 
getWeldType(colType));
                if (count > 0 || dep != NULL) {
                        sprintf(structMember, ".$%d", count);
                } else {
                        structMember[0] = '\0';
                }
                if (getBatType(colType) == TYPE_str) {
-                       sprintf(dictKeyStmt + strlen(dictKeyStmt),
-                                       " strslice(v%dstr, i64(n%s) + 
v%dstroffset),", col, structMember, col);
+                       sprintf(dictTypeStmt + strlen(dictTypeStmt), " i64,");
+                       sprintf(dictKeyStmt + strlen(dictKeyStmt), " 
i64(n%s),", structMember);
                } else {
+                       sprintf(dictTypeStmt + strlen(dictTypeStmt), " %s,", 
getWeldType(colType));
                        sprintf(dictKeyStmt + strlen(dictKeyStmt), " n%s,", 
structMember);
                }
                ++count;
_______________________________________________
checkin-list mailing list
checkin-list@monetdb.org
https://www.monetdb.org/mailman/listinfo/checkin-list