Fix planning bug introduced in recent code reorganization for hashed

aggregates: tuple_fraction has to be adjusted before passing it to
compare_fractional_path_costs().

diff --git a/src/backend/optimizer/plan/planner.c b/src/backend/optimizer/plan/planner.c
index 7e722d6a099fd61205136ab45eeff863f2831556..ab51f0cedbb2cf8c39c39c1925d70683e7791bad 100644 (file)
--- a/src/backend/optimizer/plan/planner.c
+++ b/src/backend/optimizer/plan/planner.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/optimizer/plan/planner.c,v 1.127 2002/11/06 22:31:24 tgl Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/optimizer/plan/planner.c,v 1.128 2002/11/14 19:00:36 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1257,6 +1257,9 @@ grouping_planner(Query *parse, double tuple_fraction)
                      sorted_path->parent->rows, sorted_path->parent->width);
            sort_path.startup_cost += cheapest_path->total_cost;
            sort_path.total_cost += cheapest_path->total_cost;
+           /* Convert absolute-count tuple_fraction into a fraction */
+           if (tuple_fraction >= 1.0)
+               tuple_fraction /= sorted_path->parent->rows;
            if (compare_fractional_path_costs(sorted_path, &sort_path,
                                              tuple_fraction) <= 0)
            {