diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2003-09-07 15:26:54 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2003-09-07 15:26:54 +0000 |
| commit | 48beecda7cbd2e73ebc11c88d3567ad064604bda (patch) | |
| tree | bbe6fd0b76dc6ac7f18cefa3bb86ce19eec5c8b5 /src/include | |
| parent | 75c35e0f313d58fbbc65169dec13ce19e9886c99 (diff) | |
| download | postgresql-48beecda7cbd2e73ebc11c88d3567ad064604bda.tar.gz postgresql-48beecda7cbd2e73ebc11c88d3567ad064604bda.zip | |
Remove geqo_random_seed parameter. Having geqo reset the global random()
sequence every time it's called is bogus --- it interferes with user
control over the seed, and actually decreases randomness overall
(because a seed based on time(NULL) is pretty predictable). If you really
want a reproducible result from geqo, do 'set seed = 0' before planning
a query.
Diffstat (limited to 'src/include')
| -rw-r--r-- | src/include/optimizer/geqo.h | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/include/optimizer/geqo.h b/src/include/optimizer/geqo.h index 0ead9dc304f..53a2334a171 100644 --- a/src/include/optimizer/geqo.h +++ b/src/include/optimizer/geqo.h @@ -6,7 +6,7 @@ * Portions Copyright (c) 1996-2003, PostgreSQL Global Development Group * Portions Copyright (c) 1994, Regents of the University of California * - * $Id: geqo.h,v 1.31 2003/08/04 02:40:13 momjian Exp $ + * $Id: geqo.h,v 1.32 2003/09/07 15:26:54 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -64,8 +64,6 @@ extern double Geqo_selection_bias; #define MIN_GEQO_SELECTION_BIAS 1.5 #define MAX_GEQO_SELECTION_BIAS 2.0 -extern int Geqo_random_seed; /* or negative to use current time */ - /* routines in geqo_main.c */ extern RelOptInfo *geqo(Query *root, int number_of_rels, List *initial_rels); |