Changeset: d2e3331ae1f1 for MonetDB
URL: https://dev.monetdb.org/hg/MonetDB/rev/d2e3331ae1f1
Modified Files:
        sql/server/rel_statistics_functions.c
        sql/test/BugTracker-2024/Tests/All
Branch: Dec2023
Log Message:

fixed bug #7458, ie fixed property propagation with sign function.


diffs (24 lines):

diff --git a/sql/server/rel_statistics_functions.c 
b/sql/server/rel_statistics_functions.c
--- a/sql/server/rel_statistics_functions.c
+++ b/sql/server/rel_statistics_functions.c
@@ -409,7 +409,11 @@ sql_sign_propagate_statistics(mvc *sql, 
                atom *zero1 = atom_zero_value(sql->sa, &(omin->tpe));
                int cmp1 = atom_cmp(omax, zero1), cmp2 = atom_cmp(omin, zero1);
 
-               if (cmp1 >= 0 && cmp2 >= 0) {
+               if (cmp1 == 0 && cmp2 == 0) {
+                       set_minmax_property(sql, e, PROP_MAX, atom_int(sql->sa, 
bte, 0));
+                       set_minmax_property(sql, e, PROP_MIN, atom_int(sql->sa, 
bte, 0));
+                       properties_set = true;
+               } else if (cmp1 > 0 && cmp2 > 0) {
                        set_minmax_property(sql, e, PROP_MAX, atom_int(sql->sa, 
bte, 1));
                        set_minmax_property(sql, e, PROP_MIN, atom_int(sql->sa, 
bte, 1));
                        properties_set = true;
diff --git a/sql/test/BugTracker-2024/Tests/All 
b/sql/test/BugTracker-2024/Tests/All
--- a/sql/test/BugTracker-2024/Tests/All
+++ b/sql/test/BugTracker-2024/Tests/All
@@ -24,3 +24,4 @@ between-boolean-7451
 concat_order-7455
 order-by-false-or-false-Bug-7456
 constant-expression-issue-7457
+sign-issue-7458
_______________________________________________
checkin-list mailing list -- checkin-list@monetdb.org
To unsubscribe send an email to checkin-list-le...@monetdb.org

Reply via email to