MonetDB: Dec2023 - solved optimzer issue fixing bug #7468

[Niels Nes via checkin-list]

Changeset: dcb9e985f2d5 for MonetDB
URL: https://dev.monetdb.org/hg/MonetDB/rev/dcb9e985f2d5
Modified Files:
        sql/server/rel_statistics_functions.c
Branch: Dec2023
Log Message:

solved optimzer issue fixing bug #7468


diffs (26 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
@@ -191,6 +191,7 @@ sql_sub_propagate_statistics(mvc *sql, s
                }
 
                if (res1 && res2) { /* if the min/max pair overflows, then 
don't propagate */
+#if 0
                        atom *zero1 = atom_zero_value(sql->sa, &(lmax->tpe)), 
*zero2 = atom_zero_value(sql->sa, &(rmax->tpe));
                        int cmp1 = atom_cmp(lmax, zero1), cmp2 = atom_cmp(lmin, 
zero1), cmp3 = atom_cmp(rmin, zero2), cmp4 = atom_cmp(rmax, zero2);
 
@@ -201,6 +202,14 @@ sql_sub_propagate_statistics(mvc *sql, s
                                set_minmax_property(sql, e, PROP_MAX, res2);
                                set_minmax_property(sql, e, PROP_MIN, res1);
                        }
+#endif
+                       if (atom_cmp(res1, res2) > 0) {
+                               set_minmax_property(sql, e, PROP_MAX, res1);
+                               set_minmax_property(sql, e, PROP_MIN, res2);
+                       } else {
+                               set_minmax_property(sql, e, PROP_MAX, res2);
+                               set_minmax_property(sql, e, PROP_MIN, res1);
+                       }
                }
                freeException(msg1);
                freeException(msg2);
_______________________________________________
checkin-list mailing list -- checkin-list@monetdb.org
To unsubscribe send an email to checkin-list-le...@monetdb.org