Rename auto_explain.sample_ratio to sample_rate

Per suggestion from Tomas Vondra

Author: Julien Rouhaud

diff --git a/contrib/auto_explain/auto_explain.c b/contrib/auto_explain/auto_explain.c
index 6708d817fba43719663adb17e6a2321a4cd59053..fcd11a00713c828e5a9c128af24bcede859bb324 100644 (file)
--- a/contrib/auto_explain/auto_explain.c
+++ b/contrib/auto_explain/auto_explain.c
@@ -29,7 +29,7 @@ static bool auto_explain_log_triggers = false;
 static bool auto_explain_log_timing = true;
 static int auto_explain_log_format = EXPLAIN_FORMAT_TEXT;
 static bool auto_explain_log_nested_statements = false;
-static double auto_explain_sample_ratio = 1;
+static double auto_explain_sample_rate = 1;
 
 static const struct config_enum_entry format_options[] = {
    {"text", EXPLAIN_FORMAT_TEXT, false},
@@ -163,10 +163,10 @@ _PG_init(void)
                             NULL,
                             NULL);
 
-   DefineCustomRealVariable("auto_explain.sample_ratio",
+   DefineCustomRealVariable("auto_explain.sample_rate",
                             "Fraction of queries to process.",
                            NULL,
-                           &auto_explain_sample_ratio,
+                           &auto_explain_sample_rate,
                            1.0,
                            0.0,
                            1.0,
@@ -209,11 +209,11 @@ static void
 explain_ExecutorStart(QueryDesc *queryDesc, int eflags)
 {
    /*
-    * For ratio sampling, randomly choose top-level statement. Either
+    * For rate sampling, randomly choose top-level statement. Either
     * all nested statements will be explained or none will.
     */
    if (auto_explain_log_min_duration >= 0 && nesting_level == 0)
-       current_query_sampled = (random() < auto_explain_sample_ratio *
+       current_query_sampled = (random() < auto_explain_sample_rate *
                MAX_RANDOM_VALUE);
 
    if (auto_explain_enabled() && current_query_sampled)

diff --git a/doc/src/sgml/auto-explain.sgml b/doc/src/sgml/auto-explain.sgml
index 6f1bde0d17ef2a00bbfc46573a3e5b47f87e82fb..38e6f50c8029b0249d18b2f702b27365056564e1 100644 (file)
--- a/doc/src/sgml/auto-explain.sgml
+++ b/doc/src/sgml/auto-explain.sgml
@@ -206,18 +206,17 @@ LOAD 'auto_explain';
 
    
     
-     auto_explain.sample_ratio (real)
+     auto_explain.sample_rate (real)
      
-      auto_explain.sample_ratio configuration parameter
+      auto_explain.sample_rate configuration parameter
      
     
     
      
-      auto_explain.sample_ratio (floating point)
-      causes auto_explain to only explain a fraction of the statements in each
-      session.  The default is 1, meaning explain all the queries.  In case
-      of nested statements, either all will be explained or none. Only
-      superusers can change this setting.
+      auto_explain.sample_rate causes auto_explain to only
+      explain a fraction of the statements in each session.  The default is 1,
+      meaning explain all the queries.  In case of nested statements, either all
+      will be explained or none. Only superusers can change this setting.