Changeset: fd95290f0733 for MonetDB
URL: https://dev.monetdb.org/hg/MonetDB/rev/fd95290f0733
Modified Files:
        sql/backends/monet5/dict.c
        sql/backends/monet5/for.c
        sql/server/rel_rel.c
Branch: default
Log Message:

Typo and cleanup


diffs (86 lines):

diff --git a/sql/backends/monet5/dict.c b/sql/backends/monet5/dict.c
--- a/sql/backends/monet5/dict.c
+++ b/sql/backends/monet5/dict.c
@@ -237,7 +237,7 @@ DICTcompress_col(Client cntxt, MalBlkPtr
        if (!c)
                throw(SQL, "dict.compress", SQLSTATE(3F000) "column '%s.%s.%s' 
unknown", sname, tname, cname);
        if (c->storage_type)
-               throw(SQL, "dict.compress", SQLSTATE(3F000) "column '%s.%s.%s' 
allready compressed", sname, tname, cname);
+               throw(SQL, "dict.compress", SQLSTATE(3F000) "column '%s.%s.%s' 
already compressed", sname, tname, cname);
 
        sqlstore *store = tr->store;
        BAT *b = store->storage_api.bind_col(tr, c, RDONLY), *o, *u;
diff --git a/sql/backends/monet5/for.c b/sql/backends/monet5/for.c
--- a/sql/backends/monet5/for.c
+++ b/sql/backends/monet5/for.c
@@ -42,7 +42,7 @@ FORdecompress(Client cntxt, MalBlkPtr mb
                        tt != TYPE_hge &&
 #endif
                        tt != TYPE_lng && tt != TYPE_int)
-               throw(SQL, "for.decompress", SQLSTATE(3F000) "for decompress: 
invalid type");
+               throw(SQL, "for.decompress", SQLSTATE(3F000) "for decompress: 
invalid offset type");
 
        BAT *o = BATdescriptor(O), *b = NULL;
        if (!o) {
@@ -76,7 +76,7 @@ FORdecompress(Client cntxt, MalBlkPtr mb
                }
        } else {
                bat_destroy(o);
-               throw(SQL, "for.decompress", SQLSTATE(3F000) "unknown offset 
type");
+               throw(SQL, "for.decompress", SQLSTATE(3F000) "offset type %s 
not yet implemented", ATOMname(tt));
        }
        bat_destroy(o);
        BATsetcount(b, cnt);
@@ -109,14 +109,7 @@ FORcompress_intern(char **comp_min_val, 
        }
 
        BUN cnt = BATcount(b);
-#ifdef HAVE_HGE
-       if (b->ttype == TYPE_hge) {
-               GDKfree(mn);
-               GDKfree(mx);
-               throw(SQL, "for.compress", SQLSTATE(3F000) "for compress: 
implement type hge");
-       } else
-#endif
-       if (b->ttype == TYPE_lng) {
+       if (tt == TYPE_lng) {
                lng min_val = *(lng*)mn;
                lng max_val = *(lng*)mx;
                GDKfree(mn);
@@ -141,14 +134,10 @@ FORcompress_intern(char **comp_min_val, 
                                ov[i] = (sht)(iv[i] - min_val);
                }
                snprintf(buf, 64, "FOR-" LLFMT, min_val);
-       } else if (b->ttype == TYPE_int) {
-               GDKfree(mn);
-               GDKfree(mx);
-               throw(SQL, "for.compress", SQLSTATE(3F000) "for compress: 
implement type int");
        } else {
                GDKfree(mn);
                GDKfree(mx);
-               throw(SQL, "for.compress", SQLSTATE(3F000) "for compress: type 
not yet implemented");
+               throw(SQL, "for.compress", SQLSTATE(3F000) "for compress: type 
%s not yet implemented", ATOMname(tt));
        }
        if (!(*comp_min_val = GDKstrdup(buf))) {
                bat_destroy(o);
@@ -199,7 +188,7 @@ FORcompress_col(Client cntxt, MalBlkPtr 
        if (c->null)
                throw(SQL, "for.compress", SQLSTATE(3F000) "for compress: for 
'for' compression column's cannot have NULL's");
        if (c->storage_type)
-               throw(SQL, "for.compress", SQLSTATE(3F000) "column '%s.%s.%s' 
allready compressed", sname, tname, cname);
+               throw(SQL, "for.compress", SQLSTATE(3F000) "column '%s.%s.%s' 
already compressed", sname, tname, cname);
 
        sqlstore *store = tr->store;
        BAT *b = store->storage_api.bind_col(tr, c, RDONLY), *o = NULL;
diff --git a/sql/server/rel_rel.c b/sql/server/rel_rel.c
--- a/sql/server/rel_rel.c
+++ b/sql/server/rel_rel.c
@@ -1720,7 +1720,7 @@ rel_deps(mvc *sql, sql_rel *r, list *ref
        if (!r)
                return 0;
 
-       if (rel_is_ref(r) && refs_find_rel(refs, r)) /* allready handled */
+       if (rel_is_ref(r) && refs_find_rel(refs, r)) /* already handled */
                return 0;
        switch (r->op) {
        case op_basetable: {
_______________________________________________
checkin-list mailing list
checkin-list@monetdb.org
https://www.monetdb.org/mailman/listinfo/checkin-list

Reply via email to