diff --git a/doc/src/sgml/config.sgml b/doc/src/sgml/config.sgml index fa8ce23a46..8658f493d5 100644 --- a/doc/src/sgml/config.sgml +++ b/doc/src/sgml/config.sgml @@ -1,4 +1,4 @@ - + Server Configuration @@ -4187,20 +4187,6 @@ SET XML OPTION { DOCUMENT | CONTENT }; - - explain_pretty_print (boolean) - - explain_pretty_print configuration parameter - - - - Determines whether EXPLAIN VERBOSE uses the - indented or non-indented format for displaying detailed - query-tree dumps. The default is on. - - - - dynamic_library_path (string) diff --git a/doc/src/sgml/ref/explain.sgml b/doc/src/sgml/ref/explain.sgml index 89fb0274ca..73cea6b00f 100644 --- a/doc/src/sgml/ref/explain.sgml +++ b/doc/src/sgml/ref/explain.sgml @@ -1,5 +1,5 @@ @@ -106,12 +106,7 @@ ROLLBACK; VERBOSE - Show the full internal representation of the plan tree, rather - than just a summary. Usually this option is only useful for - specialized debugging purposes. The - VERBOSE output is either pretty-printed or - not, depending on the setting of the configuration parameter. + Include the output column list for each node in the plan tree. diff --git a/src/backend/commands/explain.c b/src/backend/commands/explain.c index 768ae7797e..cbcc2a67e1 100644 --- a/src/backend/commands/explain.c +++ b/src/backend/commands/explain.c @@ -7,7 +7,7 @@ * Portions Copyright (c) 1994-5, Regents of the University of California * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/commands/explain.c,v 1.172 2008/04/17 18:30:18 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/commands/explain.c,v 1.173 2008/04/18 01:42:17 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -20,7 +20,6 @@ #include "commands/prepare.h" #include "commands/trigger.h" #include "executor/instrument.h" -#include "nodes/print.h" #include "optimizer/clauses.h" #include "optimizer/planner.h" #include "optimizer/var.h" @@ -44,7 +43,7 @@ explain_get_index_name_hook_type explain_get_index_name_hook = NULL; typedef struct ExplainState { /* options */ - bool printNodes; /* do nodeToString() too */ + bool printTList; /* print plan targetlists */ bool printAnalyze; /* print actual times */ /* other states */ PlannedStmt *pstmt; /* top of plan */ @@ -271,30 +270,11 @@ ExplainOnePlan(PlannedStmt *plannedstmt, ParamListInfo params, es = (ExplainState *) palloc0(sizeof(ExplainState)); - es->printNodes = stmt->verbose; + es->printTList = stmt->verbose; es->printAnalyze = stmt->analyze; es->pstmt = queryDesc->plannedstmt; es->rtable = queryDesc->plannedstmt->rtable; - if (es->printNodes) - { - char *s; - char *f; - - s = nodeToString(queryDesc->plannedstmt->planTree); - if (s) - { - if (Explain_pretty_print) - f = pretty_format_node_dump(s); - else - f = format_node_dump(s); - pfree(s); - do_text_output_multiline(tstate, f); - pfree(f); - do_text_output_oneline(tstate, ""); /* separator line */ - } - } - initStringInfo(&buf); explain_outNode(&buf, queryDesc->plannedstmt->planTree, queryDesc->planstate, @@ -747,7 +727,8 @@ explain_outNode(StringInfo str, appendStringInfoChar(str, '\n'); /* target list */ - show_plan_tlist(plan, str, indent, es); + if (es->printTList) + show_plan_tlist(plan, str, indent, es); /* quals, sort keys, etc */ switch (nodeTag(plan)) @@ -1055,7 +1036,6 @@ static void show_plan_tlist(Plan *plan, StringInfo str, int indent, ExplainState *es) { -#ifdef EXPLAIN_PRINT_TLISTS List *context; bool useprefix; ListCell *lc; @@ -1095,7 +1075,6 @@ show_plan_tlist(Plan *plan, } appendStringInfoChar(str, '\n'); -#endif /* EXPLAIN_PRINT_TLISTS */ } /* diff --git a/src/backend/utils/misc/guc.c b/src/backend/utils/misc/guc.c index 15e5d54e41..a2faea9782 100644 --- a/src/backend/utils/misc/guc.c +++ b/src/backend/utils/misc/guc.c @@ -10,7 +10,7 @@ * Written by Peter Eisentraut . * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/utils/misc/guc.c,v 1.446 2008/04/04 17:25:23 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/utils/misc/guc.c,v 1.447 2008/04/18 01:42:17 tgl Exp $ * *-------------------------------------------------------------------- */ @@ -282,7 +282,6 @@ bool Debug_print_plan = false; bool Debug_print_parse = false; bool Debug_print_rewritten = false; bool Debug_pretty_print = false; -bool Explain_pretty_print = true; bool log_parser_stats = false; bool log_planner_stats = false; @@ -807,15 +806,6 @@ static struct config_bool ConfigureNamesBool[] = }, #endif - { - {"explain_pretty_print", PGC_USERSET, CLIENT_CONN_OTHER, - gettext_noop("Uses the indented output format for EXPLAIN VERBOSE."), - NULL - }, - &Explain_pretty_print, - true, NULL, NULL - }, - { {"track_activities", PGC_SUSET, STATS_COLLECTOR, gettext_noop("Collects information about executing commands."), diff --git a/src/backend/utils/misc/postgresql.conf.sample b/src/backend/utils/misc/postgresql.conf.sample index e5ae0e8518..9fb567bd62 100644 --- a/src/backend/utils/misc/postgresql.conf.sample +++ b/src/backend/utils/misc/postgresql.conf.sample @@ -447,7 +447,6 @@ # - Other Defaults - -#explain_pretty_print = on #dynamic_library_path = '$libdir' #local_preload_libraries = '' diff --git a/src/include/utils/guc.h b/src/include/utils/guc.h index b0fb369f85..b6103550ad 100644 --- a/src/include/utils/guc.h +++ b/src/include/utils/guc.h @@ -7,7 +7,7 @@ * Copyright (c) 2000-2008, PostgreSQL Global Development Group * Written by Peter Eisentraut . * - * $PostgreSQL: pgsql/src/include/utils/guc.h,v 1.93 2008/04/02 14:42:56 mha Exp $ + * $PostgreSQL: pgsql/src/include/utils/guc.h,v 1.94 2008/04/18 01:42:17 tgl Exp $ *-------------------------------------------------------------------- */ #ifndef GUC_H @@ -127,7 +127,6 @@ extern bool Debug_print_plan; extern bool Debug_print_parse; extern bool Debug_print_rewritten; extern bool Debug_pretty_print; -extern bool Explain_pretty_print; extern bool log_parser_stats; extern bool log_planner_stats;