Changeset: 1b7bed08822f for MonetDB
URL: https://dev.monetdb.org/hg/MonetDB/rev/1b7bed08822f
Added Files:
        sql/test/miscellaneous/Tests/decimal-atoms.test
Modified Files:
        sql/server/sql_atom.c
        sql/test/miscellaneous/Tests/All
Branch: Aug2024
Log Message:

Fixes atoms' decimal representation


diffs (60 lines):

diff --git a/sql/server/sql_atom.c b/sql/server/sql_atom.c
--- a/sql/server/sql_atom.c
+++ b/sql/server/sql_atom.c
@@ -99,8 +99,6 @@ atom_int( allocator *sa, sql_subtype *tp
                int bits = number_bits(val);
                if (a->tpe.type->eclass == EC_NUM)
                        a->tpe.digits = bits;
-               else if (a->tpe.type->eclass == EC_DEC)
-                       a->tpe.digits = bits2digits(bits) + 1;;
                a->data.len = 0;
                return a;
        }
diff --git a/sql/test/miscellaneous/Tests/All b/sql/test/miscellaneous/Tests/All
--- a/sql/test/miscellaneous/Tests/All
+++ b/sql/test/miscellaneous/Tests/All
@@ -31,3 +31,4 @@ anti_join_plan
 table_alias_on_cte
 special_character_names
 group_by_all
+decimal-atoms
diff --git a/sql/test/miscellaneous/Tests/decimal-atoms.test 
b/sql/test/miscellaneous/Tests/decimal-atoms.test
new file mode 100644
--- /dev/null
+++ b/sql/test/miscellaneous/Tests/decimal-atoms.test
@@ -0,0 +1,35 @@
+query T nosort
+plan select 0.001;
+----
+project (
+|  [ boolean(1) "true" as "%1"."%1" ]
+) [ decimal(4,3) "1" ]
+
+query T nosort
+plan select .001;
+----
+project (
+|  [ boolean(1) "true" as "%1"."%1" ]
+) [ decimal(3,3) "1" ]
+
+query T nosort
+plan select 100.00;
+----
+project (
+|  [ boolean(1) "true" as "%1"."%1" ]
+) [ decimal(5,2) "10000" ]
+
+query T nosort
+plan select 100.0;
+----
+project (
+|  [ boolean(1) "true" as "%1"."%1" ]
+) [ decimal(4,1) "1000" ]
+
+query T nosort
+plan select 100.;
+----
+project (
+|  [ boolean(1) "true" as "%1"."%1" ]
+) [ tinyint(7) "100" ]
+
_______________________________________________
checkin-list mailing list -- checkin-list@monetdb.org
To unsubscribe send an email to checkin-list-le...@monetdb.org

Reply via email to