diff --git a/src/backend/distributed/planner/multi_explain.c b/src/backend/distributed/planner/multi_explain.c index 0e3e90b33..b44c59f57 100644 --- a/src/backend/distributed/planner/multi_explain.c +++ b/src/backend/distributed/planner/multi_explain.c @@ -52,8 +52,6 @@ /* Config variables that enable printing distributed query plans */ -bool ExplainMultiLogicalPlan = false; -bool ExplainMultiPhysicalPlan = false; bool ExplainDistributedQueries = true; bool ExplainAllTasks = false; @@ -185,24 +183,10 @@ MultiExplainOneQuery(Query *query, IntoClause *into, ExplainState *es, INSTR_TIME_SET_CURRENT(planDuration); INSTR_TIME_SUBTRACT(planDuration, planStart); - if (ExplainMultiLogicalPlan) { - MultiTreeRoot *multiTree = MultiLogicalPlanCreate(query); - char *logicalPlanString = CitusNodeToString(multiTree); - char *formattedPlanString = pretty_format_node_dump(logicalPlanString); - appendStringInfo(es->str, "logical plan:\n"); - appendStringInfo(es->str, "%s\n", formattedPlanString); } - if (ExplainMultiPhysicalPlan) - { - char *physicalPlanString = CitusNodeToString(multiPlan); - char *formattedPlanString = pretty_format_node_dump(physicalPlanString); - - appendStringInfo(es->str, "physical plan:\n"); - appendStringInfo(es->str, "%s\n", formattedPlanString); - } if (!ExplainDistributedQueries) { diff --git a/src/backend/distributed/shared_library_init.c b/src/backend/distributed/shared_library_init.c index 509ab0381..102016c8a 100644 --- a/src/backend/distributed/shared_library_init.c +++ b/src/backend/distributed/shared_library_init.c @@ -295,30 +295,6 @@ RegisterCitusConfigVariables(void) 0, NULL, NULL, NULL); - DefineCustomBoolVariable( - "citus.explain_multi_logical_plan", - gettext_noop("Enables Explain to print out distributed logical plans."), - gettext_noop("We use this private configuration entry as a debugging aid. " - "If enabled, the Explain command prints out the optimized " - "logical plan for distributed queries."), - &ExplainMultiLogicalPlan, - false, - PGC_USERSET, - GUC_NO_SHOW_ALL, - NULL, NULL, NULL); - - DefineCustomBoolVariable( - "citus.explain_multi_physical_plan", - gettext_noop("Enables Explain to print out distributed physical plans."), - gettext_noop("We use this private configuration entry as a debugging aid. " - "If enabled, the Explain command prints out the physical " - "plan for distributed queries."), - &ExplainMultiPhysicalPlan, - false, - PGC_USERSET, - GUC_NO_SHOW_ALL, - NULL, NULL, NULL); - DefineCustomBoolVariable( "citus.explain_distributed_queries", gettext_noop("Enables Explain for distributed queries."), diff --git a/src/include/distributed/multi_explain.h b/src/include/distributed/multi_explain.h index 984b8e2ab..55f4bf75d 100644 --- a/src/include/distributed/multi_explain.h +++ b/src/include/distributed/multi_explain.h @@ -13,8 +13,6 @@ #include "executor/executor.h" /* Config variables managed via guc.c to explain distributed query plans */ -extern bool ExplainMultiLogicalPlan; -extern bool ExplainMultiPhysicalPlan; extern bool ExplainDistributedQueries; extern bool ExplainAllTasks;