* do instructions or hints to the planner using C-style block comments
* of the SQL.
*
- * Copyright (c) 2012, NIPPON TELEGRAPH AND TELEPHONE CORPORATION
+ * Copyright (c) 2012-2013, NIPPON TELEGRAPH AND TELEPHONE CORPORATION
*
*-------------------------------------------------------------------------
*/
#include "postgres.h"
+#include "catalog/pg_collation.h"
+#include "catalog/pg_index.h"
#include "commands/prepare.h"
#include "mb/pg_wchar.h"
#include "miscadmin.h"
#include "optimizer/restrictinfo.h"
#include "parser/scansup.h"
#include "tcop/utility.h"
+#include "utils/builtins.h"
#include "utils/lsyscache.h"
#include "utils/memutils.h"
+#include "utils/rel.h"
+#include "utils/syscache.h"
#if PG_VERSION_NUM >= 90200
#include "catalog/pg_class.h"
#endif
+#include "executor/spi.h"
+#include "catalog/pg_type.h"
+/*
+ * We have our own header file "plpgsql-9.1", which is necessary to support
+ * hints for queries in PL/pgSQL blocks, in pg_hint_plan source package,
+ * because PostgreSQL 9.1 doesn't provide the header file as a part of
+ * installation. This header file is a copy of src/pl/plpgsql/src/plpgsql.h in
+ * PostgreSQL 9.1.9 source tree,
+ *
+ * On the other hand, 9.2 installation provides that header file for external
+ * modules, so we include the header in ordinary place.
+ */
+#if PG_VERSION_NUM >= 90200
+#include "plpgsql.h"
+#else
+#include "plpgsql-9.1.h"
+#endif
+
+/* partially copied from pg_stat_statements */
+#include "normalize_query.h"
+
#ifdef PG_MODULE_MAGIC
PG_MODULE_MAGIC;
#endif
/* hint keywords */
#define HINT_SEQSCAN "SeqScan"
#define HINT_INDEXSCAN "IndexScan"
+#define HINT_INDEXSCANREGEXP "IndexScanRegexp"
#define HINT_BITMAPSCAN "BitmapScan"
+#define HINT_BITMAPSCANREGEXP "BitmapScanRegexp"
#define HINT_TIDSCAN "TidScan"
#define HINT_NOSEQSCAN "NoSeqScan"
#define HINT_NOINDEXSCAN "NoIndexScan"
#define HINT_NOTIDSCAN "NoTidScan"
#if PG_VERSION_NUM >= 90200
#define HINT_INDEXONLYSCAN "IndexOnlyScan"
+#define HINT_INDEXONLYSCANREGEXP "IndexOnlyScanRegexp"
#define HINT_NOINDEXONLYSCAN "NoIndexOnlyScan"
#endif
#define HINT_NESTLOOP "NestLoop"
#define DISABLE_ALL_SCAN 0
#define DISABLE_ALL_JOIN 0
+/* hint keyword of enum type*/
+typedef enum HintKeyword
+{
+ HINT_KEYWORD_SEQSCAN,
+ HINT_KEYWORD_INDEXSCAN,
+ HINT_KEYWORD_INDEXSCANREGEXP,
+ HINT_KEYWORD_BITMAPSCAN,
+ HINT_KEYWORD_BITMAPSCANREGEXP,
+ HINT_KEYWORD_TIDSCAN,
+ HINT_KEYWORD_NOSEQSCAN,
+ HINT_KEYWORD_NOINDEXSCAN,
+ HINT_KEYWORD_NOBITMAPSCAN,
+ HINT_KEYWORD_NOTIDSCAN,
+#if PG_VERSION_NUM >= 90200
+ HINT_KEYWORD_INDEXONLYSCAN,
+ HINT_KEYWORD_INDEXONLYSCANREGEXP,
+ HINT_KEYWORD_NOINDEXONLYSCAN,
+#endif
+ HINT_KEYWORD_NESTLOOP,
+ HINT_KEYWORD_MERGEJOIN,
+ HINT_KEYWORD_HASHJOIN,
+ HINT_KEYWORD_NONESTLOOP,
+ HINT_KEYWORD_NOMERGEJOIN,
+ HINT_KEYWORD_NOHASHJOIN,
+ HINT_KEYWORD_LEADING,
+ HINT_KEYWORD_SET,
+ HINT_KEYWORD_UNRECOGNIZED
+} HintKeyword;
+
typedef struct Hint Hint;
typedef struct HintState HintState;
typedef Hint *(*HintCreateFunction) (const char *hint_str,
- const char *keyword);
+ const char *keyword,
+ HintKeyword hint_keyword);
typedef void (*HintDeleteFunction) (Hint *hint);
-typedef void (*HintDumpFunction) (Hint *hint, StringInfo buf);
+typedef void (*HintDescFunction) (Hint *hint, StringInfo buf);
typedef int (*HintCmpFunction) (const Hint *a, const Hint *b);
typedef const char *(*HintParseFunction) (Hint *hint, HintState *hstate,
Query *parse, const char *str);
{
const char *hint_str; /* must not do pfree */
const char *keyword; /* must not do pfree */
+ HintKeyword hint_keyword;
HintType type;
HintStatus state;
HintDeleteFunction delete_func;
- HintDumpFunction dump_func;
+ HintDescFunction desc_func;
HintCmpFunction cmp_func;
HintParseFunction parse_func;
};
Hint base;
char *relname;
List *indexnames;
+ bool regexp;
unsigned char enforce_mask;
} ScanMethodHint;
+typedef struct ParentIndexInfo
+{
+ bool indisunique;
+ Oid method;
+ List *column_names;
+ char *expression_str;
+ Oid *indcollation;
+ Oid *opclass;
+ int16 *indoption;
+ char *indpred_str;
+} ParentIndexInfo;
+
/* join method hints */
typedef struct JoinMethodHint
{
Hint base;
int nrels;
+ int inner_nrels;
char **relnames;
unsigned char enforce_mask;
Relids joinrelids;
+ Relids inner_joinrelids;
} JoinMethodHint;
/* join order hints */
+typedef struct OuterInnerRels
+{
+ char *relation;
+ List *outer_inner_pair;
+} OuterInnerRels;
+
typedef struct LeadingHint
{
- Hint base;
- List *relations; /* relation names specified in Leading hint */
+ Hint base;
+ List *relations; /* relation names specified in Leading hint */
+ OuterInnerRels *outer_inner;
} LeadingHint;
/* change a run-time parameter hints */
ScanMethodHint **scan_hints; /* parsed scan hints */
int init_scan_mask; /* initial value scan parameter */
Index parent_relid; /* inherit parent table relid */
+ Oid parent_rel_oid; /* inherit parent table relid */
ScanMethodHint *parent_hint; /* inherit parent table scan hint */
+ List *parent_index_infos; /* information of inherit parent table's
+ * index */
/* for join method hints */
JoinMethodHint **join_hints; /* parsed join hints */
List **join_hint_level;
/* for Leading hint */
- LeadingHint *leading_hint; /* parsed last specified Leading hint */
+ LeadingHint **leading_hint; /* parsed Leading hints */
/* for Set hints */
SetHint **set_hints; /* parsed Set hints */
{
char *keyword;
HintCreateFunction create_func;
+ HintKeyword hint_keyword;
} HintParser;
/* Module callbacks */
int levels_needed,
List *initial_rels);
-static Hint *ScanMethodHintCreate(const char *hint_str, const char *keyword);
+static Hint *ScanMethodHintCreate(const char *hint_str, const char *keyword,
+ HintKeyword hint_keyword);
static void ScanMethodHintDelete(ScanMethodHint *hint);
-static void ScanMethodHintDump(ScanMethodHint *hint, StringInfo buf);
+static void ScanMethodHintDesc(ScanMethodHint *hint, StringInfo buf);
static int ScanMethodHintCmp(const ScanMethodHint *a, const ScanMethodHint *b);
static const char *ScanMethodHintParse(ScanMethodHint *hint, HintState *hstate,
Query *parse, const char *str);
-static Hint *JoinMethodHintCreate(const char *hint_str, const char *keyword);
+static Hint *JoinMethodHintCreate(const char *hint_str, const char *keyword,
+ HintKeyword hint_keyword);
static void JoinMethodHintDelete(JoinMethodHint *hint);
-static void JoinMethodHintDump(JoinMethodHint *hint, StringInfo buf);
+static void JoinMethodHintDesc(JoinMethodHint *hint, StringInfo buf);
static int JoinMethodHintCmp(const JoinMethodHint *a, const JoinMethodHint *b);
static const char *JoinMethodHintParse(JoinMethodHint *hint, HintState *hstate,
Query *parse, const char *str);
-static Hint *LeadingHintCreate(const char *hint_str, const char *keyword);
+static Hint *LeadingHintCreate(const char *hint_str, const char *keyword,
+ HintKeyword hint_keyword);
static void LeadingHintDelete(LeadingHint *hint);
-static void LeadingHintDump(LeadingHint *hint, StringInfo buf);
+static void LeadingHintDesc(LeadingHint *hint, StringInfo buf);
static int LeadingHintCmp(const LeadingHint *a, const LeadingHint *b);
static const char *LeadingHintParse(LeadingHint *hint, HintState *hstate,
Query *parse, const char *str);
-static Hint *SetHintCreate(const char *hint_str, const char *keyword);
+static Hint *SetHintCreate(const char *hint_str, const char *keyword,
+ HintKeyword hint_keyword);
static void SetHintDelete(SetHint *hint);
-static void SetHintDump(SetHint *hint, StringInfo buf);
+static void SetHintDesc(SetHint *hint, StringInfo buf);
static int SetHintCmp(const SetHint *a, const SetHint *b);
static const char *SetHintParse(SetHint *hint, HintState *hstate, Query *parse,
const char *str);
+static void quote_value(StringInfo buf, const char *value);
+
+static const char *parse_quoted_value(const char *str, char **word,
+ bool truncate);
+
RelOptInfo *pg_hint_plan_standard_join_search(PlannerInfo *root,
int levels_needed,
List *initial_rels);
RelOptInfo *pg_hint_plan_make_join_rel(PlannerInfo *root, RelOptInfo *rel1,
RelOptInfo *rel2);
+static void pg_hint_plan_plpgsql_stmt_beg(PLpgSQL_execstate *estate,
+ PLpgSQL_stmt *stmt);
+static void pg_hint_plan_plpgsql_stmt_end(PLpgSQL_execstate *estate,
+ PLpgSQL_stmt *stmt);
+
/* GUC variables */
static bool pg_hint_plan_enable_hint = true;
static bool pg_hint_plan_debug_print = false;
static int pg_hint_plan_parse_messages = INFO;
+/* Default is off, to keep backward compatibility. */
+static bool pg_hint_plan_enable_hint_table = false;
static const struct config_enum_entry parse_messages_level_options[] = {
{"debug", DEBUG2, true},
static char *stmt_name = NULL;
static const HintParser parsers[] = {
- {HINT_SEQSCAN, ScanMethodHintCreate},
- {HINT_INDEXSCAN, ScanMethodHintCreate},
- {HINT_BITMAPSCAN, ScanMethodHintCreate},
- {HINT_TIDSCAN, ScanMethodHintCreate},
- {HINT_NOSEQSCAN, ScanMethodHintCreate},
- {HINT_NOINDEXSCAN, ScanMethodHintCreate},
- {HINT_NOBITMAPSCAN, ScanMethodHintCreate},
- {HINT_NOTIDSCAN, ScanMethodHintCreate},
+ {HINT_SEQSCAN, ScanMethodHintCreate, HINT_KEYWORD_SEQSCAN},
+ {HINT_INDEXSCAN, ScanMethodHintCreate, HINT_KEYWORD_INDEXSCAN},
+ {HINT_INDEXSCANREGEXP, ScanMethodHintCreate, HINT_KEYWORD_INDEXSCANREGEXP},
+ {HINT_BITMAPSCAN, ScanMethodHintCreate, HINT_KEYWORD_BITMAPSCAN},
+ {HINT_BITMAPSCANREGEXP, ScanMethodHintCreate,
+ HINT_KEYWORD_BITMAPSCANREGEXP},
+ {HINT_TIDSCAN, ScanMethodHintCreate, HINT_KEYWORD_TIDSCAN},
+ {HINT_NOSEQSCAN, ScanMethodHintCreate, HINT_KEYWORD_NOSEQSCAN},
+ {HINT_NOINDEXSCAN, ScanMethodHintCreate, HINT_KEYWORD_NOINDEXSCAN},
+ {HINT_NOBITMAPSCAN, ScanMethodHintCreate, HINT_KEYWORD_NOBITMAPSCAN},
+ {HINT_NOTIDSCAN, ScanMethodHintCreate, HINT_KEYWORD_NOTIDSCAN},
#if PG_VERSION_NUM >= 90200
- {HINT_INDEXONLYSCAN, ScanMethodHintCreate},
- {HINT_NOINDEXONLYSCAN, ScanMethodHintCreate},
+ {HINT_INDEXONLYSCAN, ScanMethodHintCreate, HINT_KEYWORD_INDEXONLYSCAN},
+ {HINT_INDEXONLYSCANREGEXP, ScanMethodHintCreate,
+ HINT_KEYWORD_INDEXONLYSCANREGEXP},
+ {HINT_NOINDEXONLYSCAN, ScanMethodHintCreate, HINT_KEYWORD_NOINDEXONLYSCAN},
#endif
- {HINT_NESTLOOP, JoinMethodHintCreate},
- {HINT_MERGEJOIN, JoinMethodHintCreate},
- {HINT_HASHJOIN, JoinMethodHintCreate},
- {HINT_NONESTLOOP, JoinMethodHintCreate},
- {HINT_NOMERGEJOIN, JoinMethodHintCreate},
- {HINT_NOHASHJOIN, JoinMethodHintCreate},
- {HINT_LEADING, LeadingHintCreate},
- {HINT_SET, SetHintCreate},
- {NULL, NULL}
+ {HINT_NESTLOOP, JoinMethodHintCreate, HINT_KEYWORD_NESTLOOP},
+ {HINT_MERGEJOIN, JoinMethodHintCreate, HINT_KEYWORD_MERGEJOIN},
+ {HINT_HASHJOIN, JoinMethodHintCreate, HINT_KEYWORD_HASHJOIN},
+ {HINT_NONESTLOOP, JoinMethodHintCreate, HINT_KEYWORD_NONESTLOOP},
+ {HINT_NOMERGEJOIN, JoinMethodHintCreate, HINT_KEYWORD_NOMERGEJOIN},
+ {HINT_NOHASHJOIN, JoinMethodHintCreate, HINT_KEYWORD_NOHASHJOIN},
+ {HINT_LEADING, LeadingHintCreate, HINT_KEYWORD_LEADING},
+ {HINT_SET, SetHintCreate, HINT_KEYWORD_SET},
+ {NULL, NULL, HINT_KEYWORD_UNRECOGNIZED}
+};
+
+/*
+ * PL/pgSQL plugin for retrieving string representation of each query during
+ * function execution.
+ */
+const char *plpgsql_query_string = NULL;
+PLpgSQL_plugin plugin_funcs = {
+ NULL,
+ NULL,
+ NULL,
+ pg_hint_plan_plpgsql_stmt_beg,
+ pg_hint_plan_plpgsql_stmt_end,
+ NULL,
+ NULL,
};
+/* Current nesting depth of SPI calls, used to prevent recursive calls */
+static int nested_level = 0;
+
/*
* Module load callbacks
*/
void
_PG_init(void)
{
+ PLpgSQL_plugin **var_ptr;
+
/* Define custom GUC variables. */
DefineCustomBoolVariable("pg_hint_plan.enable_hint",
"Force planner to use plans specified in the hint comment preceding to the query.",
NULL,
NULL);
+ DefineCustomBoolVariable("pg_hint_plan.enable_hint_table",
+ "Force planner to not get hint by using table lookups.",
+ NULL,
+ &pg_hint_plan_enable_hint_table,
+ false,
+ PGC_USERSET,
+ 0,
+ NULL,
+ NULL,
+ NULL);
+
/* Install hooks. */
prev_ProcessUtility = ProcessUtility_hook;
ProcessUtility_hook = pg_hint_plan_ProcessUtility;
get_relation_info_hook = pg_hint_plan_get_relation_info;
prev_join_search = join_search_hook;
join_search_hook = pg_hint_plan_join_search;
+
+ /* setup PL/pgSQL plugin hook */
+ var_ptr = (PLpgSQL_plugin **) find_rendezvous_variable("PLpgSQL_plugin");
+ *var_ptr = &plugin_funcs;
}
/*
void
_PG_fini(void)
{
+ PLpgSQL_plugin **var_ptr;
+
/* Uninstall hooks. */
ProcessUtility_hook = prev_ProcessUtility;
planner_hook = prev_planner;
get_relation_info_hook = prev_get_relation_info;
join_search_hook = prev_join_search;
+
+ /* uninstall PL/pgSQL plugin hook */
+ var_ptr = (PLpgSQL_plugin **) find_rendezvous_variable("PLpgSQL_plugin");
+ *var_ptr = NULL;
}
/*
*/
static Hint *
-ScanMethodHintCreate(const char *hint_str, const char *keyword)
+ScanMethodHintCreate(const char *hint_str, const char *keyword,
+ HintKeyword hint_keyword)
{
ScanMethodHint *hint;
hint = palloc(sizeof(ScanMethodHint));
hint->base.hint_str = hint_str;
hint->base.keyword = keyword;
+ hint->base.hint_keyword = hint_keyword;
hint->base.type = HINT_TYPE_SCAN_METHOD;
hint->base.state = HINT_STATE_NOTUSED;
hint->base.delete_func = (HintDeleteFunction) ScanMethodHintDelete;
- hint->base.dump_func = (HintDumpFunction) ScanMethodHintDump;
+ hint->base.desc_func = (HintDescFunction) ScanMethodHintDesc;
hint->base.cmp_func = (HintCmpFunction) ScanMethodHintCmp;
hint->base.parse_func = (HintParseFunction) ScanMethodHintParse;
hint->relname = NULL;
hint->indexnames = NIL;
+ hint->regexp = false;
hint->enforce_mask = 0;
return (Hint *) hint;
}
static Hint *
-JoinMethodHintCreate(const char *hint_str, const char *keyword)
+JoinMethodHintCreate(const char *hint_str, const char *keyword,
+ HintKeyword hint_keyword)
{
JoinMethodHint *hint;
hint = palloc(sizeof(JoinMethodHint));
hint->base.hint_str = hint_str;
hint->base.keyword = keyword;
+ hint->base.hint_keyword = hint_keyword;
hint->base.type = HINT_TYPE_JOIN_METHOD;
hint->base.state = HINT_STATE_NOTUSED;
hint->base.delete_func = (HintDeleteFunction) JoinMethodHintDelete;
- hint->base.dump_func = (HintDumpFunction) JoinMethodHintDump;
+ hint->base.desc_func = (HintDescFunction) JoinMethodHintDesc;
hint->base.cmp_func = (HintCmpFunction) JoinMethodHintCmp;
hint->base.parse_func = (HintParseFunction) JoinMethodHintParse;
hint->nrels = 0;
+ hint->inner_nrels = 0;
hint->relnames = NULL;
hint->enforce_mask = 0;
hint->joinrelids = NULL;
+ hint->inner_joinrelids = NULL;
return (Hint *) hint;
}
pfree(hint->relnames[i]);
pfree(hint->relnames);
}
+
bms_free(hint->joinrelids);
+ bms_free(hint->inner_joinrelids);
pfree(hint);
}
static Hint *
-LeadingHintCreate(const char *hint_str, const char *keyword)
+LeadingHintCreate(const char *hint_str, const char *keyword,
+ HintKeyword hint_keyword)
{
LeadingHint *hint;
hint = palloc(sizeof(LeadingHint));
hint->base.hint_str = hint_str;
hint->base.keyword = keyword;
+ hint->base.hint_keyword = hint_keyword;
hint->base.type = HINT_TYPE_LEADING;
hint->base.state = HINT_STATE_NOTUSED;
hint->base.delete_func = (HintDeleteFunction)LeadingHintDelete;
- hint->base.dump_func = (HintDumpFunction) LeadingHintDump;
+ hint->base.desc_func = (HintDescFunction) LeadingHintDesc;
hint->base.cmp_func = (HintCmpFunction) LeadingHintCmp;
hint->base.parse_func = (HintParseFunction) LeadingHintParse;
hint->relations = NIL;
+ hint->outer_inner = NULL;
return (Hint *) hint;
}
return;
list_free_deep(hint->relations);
+ if (hint->outer_inner)
+ pfree(hint->outer_inner);
pfree(hint);
}
static Hint *
-SetHintCreate(const char *hint_str, const char *keyword)
+SetHintCreate(const char *hint_str, const char *keyword,
+ HintKeyword hint_keyword)
{
SetHint *hint;
hint = palloc(sizeof(SetHint));
hint->base.hint_str = hint_str;
hint->base.keyword = keyword;
+ hint->base.hint_keyword = hint_keyword;
hint->base.type = HINT_TYPE_SET;
hint->base.state = HINT_STATE_NOTUSED;
hint->base.delete_func = (HintDeleteFunction) SetHintDelete;
- hint->base.dump_func = (HintDumpFunction) SetHintDump;
+ hint->base.desc_func = (HintDescFunction) SetHintDesc;
hint->base.cmp_func = (HintCmpFunction) SetHintCmp;
hint->base.parse_func = (HintParseFunction) SetHintParse;
hint->name = NULL;
hstate->scan_hints = NULL;
hstate->init_scan_mask = 0;
hstate->parent_relid = 0;
+ hstate->parent_rel_oid = InvalidOid;
hstate->parent_hint = NULL;
+ hstate->parent_index_infos = NIL;
hstate->join_hints = NULL;
hstate->init_join_mask = 0;
hstate->join_hint_level = NULL;
hstate->all_hints[i]->delete_func(hstate->all_hints[i]);
if (hstate->all_hints)
pfree(hstate->all_hints);
+ if (hstate->parent_index_infos)
+ list_free(hstate->parent_index_infos);
}
/*
- * dump functions
+ * Copy given value into buf, with quoting with '"' if necessary.
*/
-
static void
-dump_quote_value(StringInfo buf, const char *value)
+quote_value(StringInfo buf, const char *value)
{
bool need_quote = false;
const char *str;
for (str = value; *str != '\0'; str++)
{
- if (isspace(*str) || *str == ')' || *str == '"')
+ if (isspace(*str) || *str == '(' || *str == ')' || *str == '"')
{
need_quote = true;
appendStringInfoCharMacro(buf, '"');
}
static void
-ScanMethodHintDump(ScanMethodHint *hint, StringInfo buf)
+ScanMethodHintDesc(ScanMethodHint *hint, StringInfo buf)
{
ListCell *l;
appendStringInfo(buf, "%s(", hint->base.keyword);
if (hint->relname != NULL)
{
- dump_quote_value(buf, hint->relname);
+ quote_value(buf, hint->relname);
foreach(l, hint->indexnames)
{
appendStringInfoCharMacro(buf, ' ');
- dump_quote_value(buf, (char *) lfirst(l));
+ quote_value(buf, (char *) lfirst(l));
}
}
appendStringInfoString(buf, ")\n");
}
static void
-JoinMethodHintDump(JoinMethodHint *hint, StringInfo buf)
+JoinMethodHintDesc(JoinMethodHint *hint, StringInfo buf)
{
int i;
appendStringInfo(buf, "%s(", hint->base.keyword);
if (hint->relnames != NULL)
{
- dump_quote_value(buf, hint->relnames[0]);
+ quote_value(buf, hint->relnames[0]);
for (i = 1; i < hint->nrels; i++)
{
appendStringInfoCharMacro(buf, ' ');
- dump_quote_value(buf, hint->relnames[i]);
+ quote_value(buf, hint->relnames[i]);
}
}
appendStringInfoString(buf, ")\n");
}
static void
-LeadingHintDump(LeadingHint *hint, StringInfo buf)
+OuterInnerDesc(OuterInnerRels *outer_inner, StringInfo buf)
{
- bool is_first;
- ListCell *l;
+ if (outer_inner->relation == NULL)
+ {
+ bool is_first;
+ ListCell *l;
+
+ is_first = true;
+
+ appendStringInfoCharMacro(buf, '(');
+ foreach(l, outer_inner->outer_inner_pair)
+ {
+ if (is_first)
+ is_first = false;
+ else
+ appendStringInfoCharMacro(buf, ' ');
+
+ OuterInnerDesc(lfirst(l), buf);
+ }
+ appendStringInfoCharMacro(buf, ')');
+ }
+ else
+ quote_value(buf, outer_inner->relation);
+}
+
+static void
+LeadingHintDesc(LeadingHint *hint, StringInfo buf)
+{
appendStringInfo(buf, "%s(", HINT_LEADING);
- is_first = true;
- foreach(l, hint->relations)
+ if (hint->outer_inner == NULL)
{
- if (is_first)
- is_first = false;
- else
- appendStringInfoCharMacro(buf, ' ');
+ ListCell *l;
+ bool is_first;
+
+ is_first = true;
- dump_quote_value(buf, (char *) lfirst(l));
+ foreach(l, hint->relations)
+ {
+ if (is_first)
+ is_first = false;
+ else
+ appendStringInfoCharMacro(buf, ' ');
+
+ quote_value(buf, (char *) lfirst(l));
+ }
}
+ else
+ OuterInnerDesc(hint->outer_inner, buf);
appendStringInfoString(buf, ")\n");
}
static void
-SetHintDump(SetHint *hint, StringInfo buf)
+SetHintDesc(SetHint *hint, StringInfo buf)
{
bool is_first = true;
ListCell *l;
else
appendStringInfoCharMacro(buf, ' ');
- dump_quote_value(buf, (char *) lfirst(l));
+ quote_value(buf, (char *) lfirst(l));
}
appendStringInfo(buf, ")\n");
}
+/*
+ * Append string which represents all hints in a given state to buf, with
+ * preceding title with them.
+ */
static void
-all_hint_dump(HintState *hstate, StringInfo buf, const char *title,
- HintStatus state)
+desc_hint_in_state(HintState *hstate, StringInfo buf, const char *title,
+ HintStatus state)
{
int i;
if (hstate->all_hints[i]->state != state)
continue;
- hstate->all_hints[i]->dump_func(hstate->all_hints[i], buf);
+ hstate->all_hints[i]->desc_func(hstate->all_hints[i], buf);
}
}
+/*
+ * Dump contents of given hstate to server log with log level LOG.
+ */
static void
HintStateDump(HintState *hstate)
{
initStringInfo(&buf);
appendStringInfoString(&buf, "pg_hint_plan:\n");
- all_hint_dump(hstate, &buf, "used hint", HINT_STATE_USED);
- all_hint_dump(hstate, &buf, "not used hint", HINT_STATE_NOTUSED);
- all_hint_dump(hstate, &buf, "duplication hint", HINT_STATE_DUPLICATION);
- all_hint_dump(hstate, &buf, "error hint", HINT_STATE_ERROR);
+ desc_hint_in_state(hstate, &buf, "used hint", HINT_STATE_USED);
+ desc_hint_in_state(hstate, &buf, "not used hint", HINT_STATE_NOTUSED);
+ desc_hint_in_state(hstate, &buf, "duplication hint", HINT_STATE_DUPLICATION);
+ desc_hint_in_state(hstate, &buf, "error hint", HINT_STATE_ERROR);
elog(LOG, "%s", buf.data);
* Parsed token is truncated within NAMEDATALEN-1 bytes, when truncate is true.
*/
static const char *
-parse_quote_value(const char *str, char **word, bool truncate)
+parse_quoted_value(const char *str, char **word, bool truncate)
{
StringInfoData buf;
bool in_quote;
break;
}
}
- else if (isspace(*str) || *str == ')' || *str == '"' || *str == '\0')
+ else if (isspace(*str) || *str == '(' || *str == ')' || *str == '"' ||
+ *str == '\0')
break;
appendStringInfoCharMacro(&buf, *str++);
return str;
}
+static OuterInnerRels *
+OuterInnerRelsCreate(char *name, List *outer_inner_list)
+{
+ OuterInnerRels *outer_inner;
+
+ outer_inner = palloc(sizeof(OuterInnerRels));
+ outer_inner->relation = name;
+ outer_inner->outer_inner_pair = outer_inner_list;
+
+ return outer_inner;
+}
+
+static const char *
+parse_parentheses_Leading_in(const char *str, OuterInnerRels **outer_inner)
+{
+ List *outer_inner_pair = NIL;
+
+ if ((str = skip_parenthesis(str, '(')) == NULL)
+ return NULL;
+
+ skip_space(str);
+
+ /* Store words in parentheses into outer_inner_list. */
+ while(*str != ')' && *str != '\0')
+ {
+ OuterInnerRels *outer_inner_rels;
+
+ if (*str == '(')
+ {
+ str = parse_parentheses_Leading_in(str, &outer_inner_rels);
+ if (str == NULL)
+ break;
+ }
+ else
+ {
+ char *name;
+
+ if ((str = parse_quoted_value(str, &name, true)) == NULL)
+ break;
+ else
+ outer_inner_rels = OuterInnerRelsCreate(name, NIL);
+ }
+
+ outer_inner_pair = lappend(outer_inner_pair, outer_inner_rels);
+ skip_space(str);
+ }
+
+ if (str == NULL ||
+ (str = skip_parenthesis(str, ')')) == NULL)
+ {
+ list_free(outer_inner_pair);
+ return NULL;
+ }
+
+ *outer_inner = OuterInnerRelsCreate(NULL, outer_inner_pair);
+
+ return str;
+}
+
+static const char *
+parse_parentheses_Leading(const char *str, List **name_list,
+ OuterInnerRels **outer_inner)
+{
+ char *name;
+ bool truncate = true;
+
+ if ((str = skip_parenthesis(str, '(')) == NULL)
+ return NULL;
+
+ skip_space(str);
+ if (*str =='(')
+ {
+ if ((str = parse_parentheses_Leading_in(str, outer_inner)) == NULL)
+ return NULL;
+ }
+ else
+ {
+ /* Store words in parentheses into name_list. */
+ while(*str != ')' && *str != '\0')
+ {
+ if ((str = parse_quoted_value(str, &name, truncate)) == NULL)
+ {
+ list_free(*name_list);
+ return NULL;
+ }
+
+ *name_list = lappend(*name_list, name);
+ skip_space(str);
+ }
+ }
+
+ if ((str = skip_parenthesis(str, ')')) == NULL)
+ return NULL;
+ return str;
+}
+
static const char *
-parse_parentheses(const char *str, List **name_list, HintType type)
+parse_parentheses(const char *str, List **name_list, HintKeyword keyword)
{
char *name;
bool truncate = true;
/* Store words in parentheses into name_list. */
while(*str != ')' && *str != '\0')
{
- if ((str = parse_quote_value(str, &name, truncate)) == NULL)
+ if ((str = parse_quoted_value(str, &name, truncate)) == NULL)
{
list_free(*name_list);
return NULL;
*name_list = lappend(*name_list, name);
skip_space(str);
- if (type == HINT_TYPE_SET)
+ if (keyword == HINT_KEYWORD_INDEXSCANREGEXP ||
+#if PG_VERSION_NUM >= 90200
+ keyword == HINT_KEYWORD_INDEXONLYSCANREGEXP ||
+#endif
+ keyword == HINT_KEYWORD_BITMAPSCANREGEXP ||
+ keyword == HINT_KEYWORD_SET)
{
truncate = false;
}
char *keyword = parser->keyword;
Hint *hint;
- if (strcmp(buf.data, keyword) != 0)
+ if (strcasecmp(buf.data, keyword) != 0)
continue;
- hint = parser->create_func(head, keyword);
+ hint = parser->create_func(head, keyword, parser->hint_keyword);
/* parser of each hint does parse in a parenthesis. */
if ((str = hint->parse_func(hint, hstate, parse, str)) == NULL)
pfree(buf.data);
}
+
+/*
+ * Get hints from table by client-supplied query string and application name.
+ */
+static const char *
+get_hints_from_table(const char *client_query, const char *client_application)
+{
+ const char *search_query =
+ "SELECT hints "
+ " FROM hint_plan.hints "
+ " WHERE norm_query_string = $1 "
+ " AND ( application_name = $2 "
+ " OR application_name = '' ) "
+ " ORDER BY application_name DESC";
+ static SPIPlanPtr plan = NULL;
+ char *hints = NULL;
+ Oid argtypes[2] = { TEXTOID, TEXTOID };
+ Datum values[2];
+ bool nulls[2] = { false, false };
+ text *qry;
+ text *app;
+
+ PG_TRY();
+ {
+ ++nested_level;
+
+ SPI_connect();
+
+ if (plan == NULL)
+ {
+ SPIPlanPtr p;
+ p = SPI_prepare(search_query, 2, argtypes);
+ plan = SPI_saveplan(p);
+ SPI_freeplan(p);
+ }
+
+ qry = cstring_to_text(client_query);
+ app = cstring_to_text(client_application);
+ values[0] = PointerGetDatum(qry);
+ values[1] = PointerGetDatum(app);
+
+ SPI_execute_plan(plan, values, nulls, true, 1);
+
+ if (SPI_processed > 0)
+ {
+ char *buf;
+
+ hints = SPI_getvalue(SPI_tuptable->vals[0],
+ SPI_tuptable->tupdesc, 1);
+ /*
+ * Here we use SPI_palloc to ensure that hints string is valid even
+ * after SPI_finish call. We can't use simple palloc because it
+ * allocates memory in SPI's context and that context is deleted in
+ * SPI_finish.
+ */
+ buf = SPI_palloc(strlen(hints) + 1);
+ strcpy(buf, hints);
+ hints = buf;
+ }
+
+ SPI_finish();
+
+ --nested_level;
+ }
+ PG_CATCH();
+ {
+ --nested_level;
+ PG_RE_THROW();
+ }
+ PG_END_TRY();
+
+ return hints;
+}
+
/*
- * Do basic parsing of the query head comment.
+ * Get client-supplied query string.
*/
-static HintState *
-parse_head_comment(Query *parse)
+static const char *
+get_query_string(void)
{
const char *p;
- char *head;
- char *tail;
- int len;
- int i;
- HintState *hstate;
- /* get client-supplied query string. */
if (stmt_name)
{
PreparedStatement *entry;
entry = FetchPreparedStatement(stmt_name, true);
p = entry->plansource->query_string;
}
+ else if (plpgsql_query_string)
+ p = plpgsql_query_string;
else
p = debug_query_string;
+ return p;
+}
+
+/*
+ * Get hints from the head block comment in client-supplied query string.
+ */
+static const char *
+get_hints_from_comment(const char *p)
+{
+ const char *hint_head;
+ char *head;
+ char *tail;
+ int len;
+
if (p == NULL)
return NULL;
/* extract query head comment. */
- len = strlen(HINT_START);
- skip_space(p);
- if (strncmp(p, HINT_START, len))
+ hint_head = strstr(p, HINT_START);
+ if (hint_head == NULL)
return NULL;
+ for (;p < hint_head; p++)
+ {
+ /*
+ * Allow these characters precedes hint comment:
+ * - digits
+ * - alphabets which are in ASCII range
+ * - space, tabs and new-lines
+ * - underscores, for identifier
+ * - commas, for SELECT clause, EXPLAIN and PREPARE
+ * - parentheses, for EXPLAIN and PREPARE
+ *
+ * Note that we don't use isalpha() nor isalnum() in ctype.h here to
+ * avoid behavior which depends on locale setting.
+ */
+ if (!(*p >= '0' && *p <= '9') &&
+ !(*p >= 'A' && *p <= 'Z') &&
+ !(*p >= 'a' && *p <= 'z') &&
+ !isspace(*p) &&
+ *p != '_' &&
+ *p != ',' &&
+ *p != '(' && *p != ')')
+ return NULL;
+ }
+ len = strlen(HINT_START);
head = (char *) p;
p += len;
skip_space(p);
head[len] = '\0';
p = head;
+ return p;
+}
+
+/*
+ * Parse hints that got, create hint struct from parse tree and parse hints.
+ */
+static HintState *
+create_hintstate(Query *parse, const char *hints)
+{
+ const char *p;
+ int i;
+ HintState *hstate;
+
+ if (hints == NULL)
+ return NULL;
+
+ p = hints;
hstate = HintStateCreate();
- hstate->hint_str = head;
+ hstate->hint_str = (char *) hints;
/* parse each hint. */
parse_hints(hstate, parse, p);
/*
* If an object (or a set of objects) has multiple hints of same hint-type,
- * only the last hint is valid and others are igonred in planning.
+ * only the last hint is valid and others are ignored in planning.
* Hints except the last are marked as 'duplicated' to remember the order.
*/
for (i = 0; i < hstate->nall_hints - 1; i++)
Hint *next_hint = hstate->all_hints[i + 1];
/*
+ * Leading hint is marked as 'duplicated' in transform_join_hints.
+ */
+ if (cur_hint->type == HINT_TYPE_LEADING &&
+ next_hint->type == HINT_TYPE_LEADING)
+ continue;
+
+ /*
* Note that we need to pass addresses of hint pointers, because
* HintCmp is designed to sort array of Hint* by qsort.
*/
* array which consists of all hints.
*/
hstate->scan_hints = (ScanMethodHint **) hstate->all_hints;
- hstate->join_hints = (JoinMethodHint **) hstate->all_hints +
- hstate->num_hints[HINT_TYPE_SCAN_METHOD];
- hstate->leading_hint = (LeadingHint *) hstate->all_hints[
- hstate->num_hints[HINT_TYPE_SCAN_METHOD] +
- hstate->num_hints[HINT_TYPE_JOIN_METHOD] +
- hstate->num_hints[HINT_TYPE_LEADING] - 1];
- hstate->set_hints = (SetHint **) hstate->all_hints +
- hstate->num_hints[HINT_TYPE_SCAN_METHOD] +
- hstate->num_hints[HINT_TYPE_JOIN_METHOD] +
- hstate->num_hints[HINT_TYPE_LEADING];
+ hstate->join_hints = (JoinMethodHint **) (hstate->scan_hints +
+ hstate->num_hints[HINT_TYPE_SCAN_METHOD]);
+ hstate->leading_hint = (LeadingHint **) (hstate->join_hints +
+ hstate->num_hints[HINT_TYPE_JOIN_METHOD]);
+ hstate->set_hints = (SetHint **) (hstate->leading_hint +
+ hstate->num_hints[HINT_TYPE_LEADING]);
return hstate;
}
ScanMethodHintParse(ScanMethodHint *hint, HintState *hstate, Query *parse,
const char *str)
{
- const char *keyword = hint->base.keyword;
- List *name_list = NIL;
- int length;
+ const char *keyword = hint->base.keyword;
+ HintKeyword hint_keyword = hint->base.hint_keyword;
+ List *name_list = NIL;
+ int length;
- if ((str = parse_parentheses(str, &name_list, hint->base.type)) == NULL)
+ if ((str = parse_parentheses(str, &name_list, hint_keyword)) == NULL)
return NULL;
/* Parse relation name and index name(s) if given hint accepts. */
hint->indexnames = list_delete_first(name_list);
/* check whether the hint accepts index name(s). */
- if (strcmp(keyword, HINT_INDEXSCAN) != 0 &&
+ if (length != 1 &&
+ hint_keyword != HINT_KEYWORD_INDEXSCAN &&
+ hint_keyword != HINT_KEYWORD_INDEXSCANREGEXP &&
#if PG_VERSION_NUM >= 90200
- strcmp(keyword, HINT_INDEXONLYSCAN) != 0 &&
+ hint_keyword != HINT_KEYWORD_INDEXONLYSCAN &&
+ hint_keyword != HINT_KEYWORD_INDEXONLYSCANREGEXP &&
#endif
- strcmp(keyword, HINT_BITMAPSCAN) != 0 &&
- length != 1)
+ hint_keyword != HINT_KEYWORD_BITMAPSCAN &&
+ hint_keyword != HINT_KEYWORD_BITMAPSCANREGEXP)
{
hint_ereport(str,
("%s hint accepts only one relation.",
}
/* Set a bit for specified hint. */
- if (strcmp(keyword, HINT_SEQSCAN) == 0)
- hint->enforce_mask = ENABLE_SEQSCAN;
- else if (strcmp(keyword, HINT_INDEXSCAN) == 0)
- hint->enforce_mask = ENABLE_INDEXSCAN;
- else if (strcmp(keyword, HINT_BITMAPSCAN) == 0)
- hint->enforce_mask = ENABLE_BITMAPSCAN;
- else if (strcmp(keyword, HINT_TIDSCAN) == 0)
- hint->enforce_mask = ENABLE_TIDSCAN;
- else if (strcmp(keyword, HINT_NOSEQSCAN) == 0)
- hint->enforce_mask = ENABLE_ALL_SCAN ^ ENABLE_SEQSCAN;
- else if (strcmp(keyword, HINT_NOINDEXSCAN) == 0)
- hint->enforce_mask = ENABLE_ALL_SCAN ^ ENABLE_INDEXSCAN;
- else if (strcmp(keyword, HINT_NOBITMAPSCAN) == 0)
- hint->enforce_mask = ENABLE_ALL_SCAN ^ ENABLE_BITMAPSCAN;
- else if (strcmp(keyword, HINT_NOTIDSCAN) == 0)
- hint->enforce_mask = ENABLE_ALL_SCAN ^ ENABLE_TIDSCAN;
+ switch (hint_keyword)
+ {
+ case HINT_KEYWORD_SEQSCAN:
+ hint->enforce_mask = ENABLE_SEQSCAN;
+ break;
+ case HINT_KEYWORD_INDEXSCAN:
+ hint->enforce_mask = ENABLE_INDEXSCAN;
+ break;
+ case HINT_KEYWORD_INDEXSCANREGEXP:
+ hint->enforce_mask = ENABLE_INDEXSCAN;
+ hint->regexp = true;
+ break;
+ case HINT_KEYWORD_BITMAPSCAN:
+ hint->enforce_mask = ENABLE_BITMAPSCAN;
+ break;
+ case HINT_KEYWORD_BITMAPSCANREGEXP:
+ hint->enforce_mask = ENABLE_BITMAPSCAN;
+ hint->regexp = true;
+ break;
+ case HINT_KEYWORD_TIDSCAN:
+ hint->enforce_mask = ENABLE_TIDSCAN;
+ break;
+ case HINT_KEYWORD_NOSEQSCAN:
+ hint->enforce_mask = ENABLE_ALL_SCAN ^ ENABLE_SEQSCAN;
+ break;
+ case HINT_KEYWORD_NOINDEXSCAN:
+ hint->enforce_mask = ENABLE_ALL_SCAN ^ ENABLE_INDEXSCAN;
+ break;
+ case HINT_KEYWORD_NOBITMAPSCAN:
+ hint->enforce_mask = ENABLE_ALL_SCAN ^ ENABLE_BITMAPSCAN;
+ break;
+ case HINT_KEYWORD_NOTIDSCAN:
+ hint->enforce_mask = ENABLE_ALL_SCAN ^ ENABLE_TIDSCAN;
+ break;
#if PG_VERSION_NUM >= 90200
- else if (strcmp(keyword, HINT_INDEXONLYSCAN) == 0)
- hint->enforce_mask = ENABLE_INDEXSCAN | ENABLE_INDEXONLYSCAN;
- else if (strcmp(keyword, HINT_NOINDEXONLYSCAN) == 0)
- hint->enforce_mask = ENABLE_ALL_SCAN ^ ENABLE_INDEXONLYSCAN;
+ case HINT_KEYWORD_INDEXONLYSCAN:
+ hint->enforce_mask = ENABLE_INDEXSCAN | ENABLE_INDEXONLYSCAN;
+ break;
+ case HINT_KEYWORD_INDEXONLYSCANREGEXP:
+ hint->enforce_mask = ENABLE_INDEXSCAN | ENABLE_INDEXONLYSCAN;
+ hint->regexp = true;
+ break;
+ case HINT_KEYWORD_NOINDEXONLYSCAN:
+ hint->enforce_mask = ENABLE_ALL_SCAN ^ ENABLE_INDEXONLYSCAN;
+ break;
#endif
- else
- {
- hint_ereport(str, ("Unrecognized hint keyword \"%s\".", keyword));
- return NULL;
+ default:
+ hint_ereport(str, ("Unrecognized hint keyword \"%s\".", keyword));
+ return NULL;
+ break;
}
return str;
JoinMethodHintParse(JoinMethodHint *hint, HintState *hstate, Query *parse,
const char *str)
{
- const char *keyword = hint->base.keyword;
- List *name_list = NIL;
+ const char *keyword = hint->base.keyword;
+ HintKeyword hint_keyword = hint->base.hint_keyword;
+ List *name_list = NIL;
- if ((str = parse_parentheses(str, &name_list, hint->base.type)) == NULL)
+ if ((str = parse_parentheses(str, &name_list, hint_keyword)) == NULL)
return NULL;
hint->nrels = list_length(name_list);
/* Sort hints in alphabetical order of relation names. */
qsort(hint->relnames, hint->nrels, sizeof(char *), RelnameCmp);
- if (strcmp(keyword, HINT_NESTLOOP) == 0)
- hint->enforce_mask = ENABLE_NESTLOOP;
- else if (strcmp(keyword, HINT_MERGEJOIN) == 0)
- hint->enforce_mask = ENABLE_MERGEJOIN;
- else if (strcmp(keyword, HINT_HASHJOIN) == 0)
- hint->enforce_mask = ENABLE_HASHJOIN;
- else if (strcmp(keyword, HINT_NONESTLOOP) == 0)
- hint->enforce_mask = ENABLE_ALL_JOIN ^ ENABLE_NESTLOOP;
- else if (strcmp(keyword, HINT_NOMERGEJOIN) == 0)
- hint->enforce_mask = ENABLE_ALL_JOIN ^ ENABLE_MERGEJOIN;
- else if (strcmp(keyword, HINT_NOHASHJOIN) == 0)
- hint->enforce_mask = ENABLE_ALL_JOIN ^ ENABLE_HASHJOIN;
- else
+ switch (hint_keyword)
{
- hint_ereport(str, ("Unrecognized hint keyword \"%s\".", keyword));
- return NULL;
+ case HINT_KEYWORD_NESTLOOP:
+ hint->enforce_mask = ENABLE_NESTLOOP;
+ break;
+ case HINT_KEYWORD_MERGEJOIN:
+ hint->enforce_mask = ENABLE_MERGEJOIN;
+ break;
+ case HINT_KEYWORD_HASHJOIN:
+ hint->enforce_mask = ENABLE_HASHJOIN;
+ break;
+ case HINT_KEYWORD_NONESTLOOP:
+ hint->enforce_mask = ENABLE_ALL_JOIN ^ ENABLE_NESTLOOP;
+ break;
+ case HINT_KEYWORD_NOMERGEJOIN:
+ hint->enforce_mask = ENABLE_ALL_JOIN ^ ENABLE_MERGEJOIN;
+ break;
+ case HINT_KEYWORD_NOHASHJOIN:
+ hint->enforce_mask = ENABLE_ALL_JOIN ^ ENABLE_HASHJOIN;
+ break;
+ default:
+ hint_ereport(str, ("Unrecognized hint keyword \"%s\".", keyword));
+ return NULL;
+ break;
}
return str;
}
+static bool
+OuterInnerPairCheck(OuterInnerRels *outer_inner)
+{
+ ListCell *l;
+ if (outer_inner->outer_inner_pair == NIL)
+ {
+ if (outer_inner->relation)
+ return true;
+ else
+ return false;
+ }
+
+ if (list_length(outer_inner->outer_inner_pair) == 2)
+ {
+ foreach(l, outer_inner->outer_inner_pair)
+ {
+ if (!OuterInnerPairCheck(lfirst(l)))
+ return false;
+ }
+ }
+ else
+ return false;
+
+ return true;
+}
+
+static List *
+OuterInnerList(OuterInnerRels *outer_inner)
+{
+ List *outer_inner_list = NIL;
+ ListCell *l;
+ OuterInnerRels *outer_inner_rels;
+
+ foreach(l, outer_inner->outer_inner_pair)
+ {
+ outer_inner_rels = (OuterInnerRels *)(lfirst(l));
+
+ if (outer_inner_rels->relation != NULL)
+ outer_inner_list = lappend(outer_inner_list,
+ outer_inner_rels->relation);
+ else
+ outer_inner_list = list_concat(outer_inner_list,
+ OuterInnerList(outer_inner_rels));
+ }
+ return outer_inner_list;
+}
+
static const char *
LeadingHintParse(LeadingHint *hint, HintState *hstate, Query *parse,
const char *str)
{
- List *name_list = NIL;
+ List *name_list = NIL;
+ OuterInnerRels *outer_inner = NULL;
- if ((str = parse_parentheses(str, &name_list, hint->base.type)) == NULL)
+ if ((str = parse_parentheses_Leading(str, &name_list, &outer_inner)) ==
+ NULL)
return NULL;
+ if (outer_inner != NULL)
+ name_list = OuterInnerList(outer_inner);
+
hint->relations = name_list;
+ hint->outer_inner = outer_inner;
/* A Leading hint requires at least two relations */
- if (list_length(hint->relations) < 2)
+ if ( hint->outer_inner == NULL && list_length(hint->relations) < 2)
{
hint_ereport(hint->base.hint_str,
("%s hint requires at least two relations.",
HINT_LEADING));
hint->base.state = HINT_STATE_ERROR;
}
+ else if (hint->outer_inner != NULL &&
+ !OuterInnerPairCheck(hint->outer_inner))
+ {
+ hint_ereport(hint->base.hint_str,
+ ("%s hint requires two sets of relations when parentheses nests.",
+ HINT_LEADING));
+ hint->base.state = HINT_STATE_ERROR;
+ }
return str;
}
{
List *name_list = NIL;
- if ((str = parse_parentheses(str, &name_list, hint->base.type)) == NULL)
+ if ((str = parse_parentheses(str, &name_list, hint->base.hint_keyword))
+ == NULL)
return NULL;
hint->words = name_list;
{
Node *node;
- if (!pg_hint_plan_enable_hint)
+ /*
+ * Use standard planner if pg_hint_plan is disabled or current nesting
+ * depth is nesting depth of SPI calls.
+ */
+ if (!pg_hint_plan_enable_hint || nested_level > 0)
{
if (prev_ProcessUtility)
(*prev_ProcessUtility) (parsetree, queryString, params,
static PlannedStmt *
pg_hint_plan_planner(Query *parse, int cursorOptions, ParamListInfo boundParams)
{
+ const char *hints = NULL;
+ const char *query;
+ char *norm_query;
+ pgssJumbleState jstate;
+ int query_len;
int save_nestlevel;
PlannedStmt *result;
HintState *hstate;
/*
- * Use standard planner if pg_hint_plan is disabled. Other hook functions
- * try to change plan with current_hint if any, so set it to NULL.
+ * Use standard planner if pg_hint_plan is disabled or current nesting
+ * depth is nesting depth of SPI calls. Other hook functions try to change
+ * plan with current_hint if any, so set it to NULL.
*/
- if (!pg_hint_plan_enable_hint)
- {
- current_hint = NULL;
+ if (!pg_hint_plan_enable_hint || nested_level > 0)
+ goto standard_planner_proc;
- if (prev_planner)
- return (*prev_planner) (parse, cursorOptions, boundParams);
- else
- return standard_planner(parse, cursorOptions, boundParams);
- }
+ /* Create hint struct from client-supplied query string. */
+ query = get_query_string();
- /* Create hint struct from parse tree. */
- hstate = parse_head_comment(parse);
+ /*
+ * Create hintstate from hint specified for the query, if any.
+ *
+ * First we lookup hint in pg_hint.hints table by normalized query string,
+ * unless pg_hint_plan.enable_hint_table is OFF.
+ * This parameter provides option to avoid overhead of table lookup during
+ * planning.
+ *
+ * If no hint was found, then we try to get hint from special query comment.
+ */
+ if (pg_hint_plan_enable_hint_table)
+ {
+ /*
+ * Search hint information which is stored for the query and the
+ * application. Query string is normalized before using in condition
+ * in order to allow fuzzy matching.
+ *
+ * XXX: normalizing code is copied from pg_stat_statements.c, so be
+ * careful when supporting PostgreSQL's version up.
+ */
+ jstate.jumble = (unsigned char *) palloc(JUMBLE_SIZE);
+ jstate.jumble_len = 0;
+ jstate.clocations_buf_size = 32;
+ jstate.clocations = (pgssLocationLen *)
+ palloc(jstate.clocations_buf_size * sizeof(pgssLocationLen));
+ jstate.clocations_count = 0;
+ JumbleQuery(&jstate, parse);
+ /*
+ * generate_normalized_query() copies exact given query_len bytes, so we
+ * add 1 byte for null-termination here. As comments on
+ * generate_normalized_query says, generate_normalized_query doesn't
+ * take care of null-terminate, but additional 1 byte ensures that '\0'
+ * byte in the source buffer to be copied into norm_query.
+ */
+ query_len = strlen(query) + 1;
+ norm_query = generate_normalized_query(&jstate,
+ query,
+ &query_len,
+ GetDatabaseEncoding());
+ hints = get_hints_from_table(norm_query, application_name);
+ elog(DEBUG1,
+ "pg_hint_plan: get_hints_from_table [%s][%s]=>[%s]",
+ norm_query, application_name, hints ? hints : "(none)");
+ }
+ if (hints == NULL)
+ hints = get_hints_from_comment(query);
+ hstate = create_hintstate(parse, hints);
/*
* Use standard planner if the statement has not valid hint. Other hook
* NULL.
*/
if (!hstate)
- {
- current_hint = NULL;
-
- if (prev_planner)
- return (*prev_planner) (parse, cursorOptions, boundParams);
- else
- return standard_planner(parse, cursorOptions, boundParams);
- }
+ goto standard_planner_proc;
/*
* Push new hint struct to the hint stack to disable previous hint context.
pop_hint();
return result;
+
+standard_planner_proc:
+ current_hint = NULL;
+ if (prev_planner)
+ return (*prev_planner) (parse, cursorOptions, boundParams);
+ else
+ return standard_planner(parse, cursorOptions, boundParams);
}
/*
return NULL;
}
+/*
+ * regexeq
+ *
+ * Returns TRUE on match, FALSE on no match.
+ *
+ * s1 --- the data to match against
+ * s2 --- the pattern
+ *
+ * Because we copy s1 to NameData, make the size of s1 less than NAMEDATALEN.
+ */
+static bool
+regexpeq(const char *s1, const char *s2)
+{
+ NameData name;
+ text *regexp;
+ Datum result;
+
+ strcpy(name.data, s1);
+ regexp = cstring_to_text(s2);
+
+ result = DirectFunctionCall2Coll(nameregexeq,
+ DEFAULT_COLLATION_OID,
+ NameGetDatum(&name),
+ PointerGetDatum(regexp));
+ return DatumGetBool(result);
+}
+
static void
-delete_indexes(ScanMethodHint *hint, RelOptInfo *rel)
+delete_indexes(ScanMethodHint *hint, RelOptInfo *rel, Oid relationObjectId)
{
ListCell *cell;
ListCell *prev;
ListCell *next;
+ StringInfoData buf;
/*
* We delete all the IndexOptInfo list and prevent you from being usable by
* other than it.
*/
prev = NULL;
+ if (pg_hint_plan_debug_print)
+ initStringInfo(&buf);
+
for (cell = list_head(rel->indexlist); cell; cell = next)
{
IndexOptInfo *info = (IndexOptInfo *) lfirst(cell);
foreach(l, hint->indexnames)
{
- if (RelnameCmp(&indexname, &lfirst(l)) == 0)
+ char *hintname = (char *) lfirst(l);
+ bool result;
+
+ if (hint->regexp)
+ result = regexpeq(indexname, hintname);
+ else
+ result = RelnameCmp(&indexname, &hintname) == 0;
+
+ if (result)
{
use_index = true;
+ if (pg_hint_plan_debug_print)
+ {
+ appendStringInfoCharMacro(&buf, ' ');
+ quote_value(&buf, indexname);
+ }
+
+ break;
+ }
+ }
+
+ /*
+ * to make the index a candidate when definition of this index is
+ * matched with the index's definition of current_hint.
+ */
+ if (OidIsValid(relationObjectId) && !use_index)
+ {
+ foreach(l, current_hint->parent_index_infos)
+ {
+ int i;
+ HeapTuple ht_idx;
+ ParentIndexInfo *p_info = (ParentIndexInfo *)lfirst(l);
+
+ /* check to match the parameter of unique */
+ if (p_info->indisunique != info->unique)
+ continue;
+
+ /* check to match the parameter of index's method */
+ if (p_info->method != info->relam)
+ continue;
+
+ /* to check to match the indexkey's configuration */
+ if ((list_length(p_info->column_names)) !=
+ info->ncolumns)
+ continue;
+
+ /* check to match the indexkey's configuration */
+ for (i = 0; i < info->ncolumns; i++)
+ {
+ char *c_attname = NULL;
+ char *p_attname = NULL;
+
+ p_attname =
+ list_nth(p_info->column_names, i);
+
+ /* both are expressions */
+ if (info->indexkeys[i] == 0 && !p_attname)
+ continue;
+
+ /* one's column is expression, the other is not */
+ if (info->indexkeys[i] == 0 || !p_attname)
+ break;
+
+ c_attname = get_attname(relationObjectId,
+ info->indexkeys[i]);
+
+ if (strcmp(p_attname, c_attname) != 0)
+ break;
+
+ if (p_info->indcollation[i] != info->indexcollations[i])
+ break;
+
+ if (p_info->opclass[i] != info->opcintype[i])
+ break;
+
+ if (((p_info->indoption[i] & INDOPTION_DESC) != 0) !=
+ info->reverse_sort[i])
+ break;
+
+ if (((p_info->indoption[i] & INDOPTION_NULLS_FIRST) != 0) !=
+ info->nulls_first[i])
+ break;
+
+ }
+
+ if (i != info->ncolumns)
+ continue;
+
+ if ((p_info->expression_str && (info->indexprs != NIL)) ||
+ (p_info->indpred_str && (info->indpred != NIL)))
+ {
+ /*
+ * Fetch the pg_index tuple by the Oid of the index
+ */
+ ht_idx = SearchSysCache1(INDEXRELID,
+ ObjectIdGetDatum(info->indexoid));
+
+ /* check to match the expression's parameter of index */
+ if (p_info->expression_str &&
+ !heap_attisnull(ht_idx, Anum_pg_index_indexprs))
+ {
+ Datum exprsDatum;
+ bool isnull;
+ Datum result;
+
+ /*
+ * to change the expression's parameter of child's
+ * index to strings
+ */
+ exprsDatum = SysCacheGetAttr(INDEXRELID, ht_idx,
+ Anum_pg_index_indexprs,
+ &isnull);
+
+ result = DirectFunctionCall2(pg_get_expr,
+ exprsDatum,
+ ObjectIdGetDatum(
+ relationObjectId));
+
+ if (strcmp(p_info->expression_str,
+ text_to_cstring(DatumGetTextP(result))) != 0)
+ {
+ /* Clean up */
+ ReleaseSysCache(ht_idx);
+
+ continue;
+ }
+ }
+
+ /* Check to match the predicate's parameter of index */
+ if (p_info->indpred_str &&
+ !heap_attisnull(ht_idx, Anum_pg_index_indpred))
+ {
+ Datum predDatum;
+ bool isnull;
+ Datum result;
+
+ /*
+ * to change the predicate's parameter of child's
+ * index to strings
+ */
+ predDatum = SysCacheGetAttr(INDEXRELID, ht_idx,
+ Anum_pg_index_indpred,
+ &isnull);
+
+ result = DirectFunctionCall2(pg_get_expr,
+ predDatum,
+ ObjectIdGetDatum(
+ relationObjectId));
+
+ if (strcmp(p_info->indpred_str,
+ text_to_cstring(DatumGetTextP(result))) != 0)
+ {
+ /* Clean up */
+ ReleaseSysCache(ht_idx);
+
+ continue;
+ }
+ }
+
+ /* Clean up */
+ ReleaseSysCache(ht_idx);
+ }
+ else if (p_info->expression_str || (info->indexprs != NIL))
+ continue;
+ else if (p_info->indpred_str || (info->indpred != NIL))
+ continue;
+
+ use_index = true;
+
+ /* to log the candidate of index */
+ if (pg_hint_plan_debug_print)
+ {
+ appendStringInfoCharMacro(&buf, ' ');
+ quote_value(&buf, indexname);
+ }
+
break;
}
}
pfree(indexname);
}
+
+ if (pg_hint_plan_debug_print)
+ {
+ char *relname;
+ StringInfoData rel_buf;
+
+ if (OidIsValid(relationObjectId))
+ relname = get_rel_name(relationObjectId);
+ else
+ relname = hint->relname;
+
+ initStringInfo(&rel_buf);
+ quote_value(&rel_buf, relname);
+
+ ereport(LOG,
+ (errmsg("available indexes for %s(%s):%s",
+ hint->base.keyword,
+ rel_buf.data,
+ buf.data)));
+ pfree(buf.data);
+ pfree(rel_buf.data);
+ }
+}
+
+/*
+ * Return information of index definition.
+ */
+static ParentIndexInfo *
+get_parent_index_info(Oid indexoid, Oid relid)
+{
+ ParentIndexInfo *p_info = palloc(sizeof(ParentIndexInfo));
+ Relation indexRelation;
+ Form_pg_index index;
+ char *attname;
+ int i;
+
+ indexRelation = index_open(indexoid, RowExclusiveLock);
+
+ index = indexRelation->rd_index;
+
+ p_info->indisunique = index->indisunique;
+ p_info->method = indexRelation->rd_rel->relam;
+
+ p_info->column_names = NIL;
+ p_info->indcollation = (Oid *) palloc(sizeof(Oid) * index->indnatts);
+ p_info->opclass = (Oid *) palloc(sizeof(Oid) * index->indnatts);
+ p_info->indoption = (int16 *) palloc(sizeof(Oid) * index->indnatts);
+
+ for (i = 0; i < index->indnatts; i++)
+ {
+ attname = get_attname(relid, index->indkey.values[i]);
+ p_info->column_names = lappend(p_info->column_names, attname);
+
+ p_info->indcollation[i] = indexRelation->rd_indcollation[i];
+ p_info->opclass[i] = indexRelation->rd_opcintype[i];
+ p_info->indoption[i] = indexRelation->rd_indoption[i];
+ }
+
+ /*
+ * to check to match the expression's parameter of index with child indexes
+ */
+ p_info->expression_str = NULL;
+ if(!heap_attisnull(indexRelation->rd_indextuple, Anum_pg_index_indexprs))
+ {
+ Datum exprsDatum;
+ bool isnull;
+ Datum result;
+
+ exprsDatum = SysCacheGetAttr(INDEXRELID, indexRelation->rd_indextuple,
+ Anum_pg_index_indexprs, &isnull);
+
+ result = DirectFunctionCall2(pg_get_expr,
+ exprsDatum,
+ ObjectIdGetDatum(relid));
+
+ p_info->expression_str = text_to_cstring(DatumGetTextP(result));
+ }
+
+ /*
+ * to check to match the predicate's parameter of index with child indexes
+ */
+ p_info->indpred_str = NULL;
+ if(!heap_attisnull(indexRelation->rd_indextuple, Anum_pg_index_indpred))
+ {
+ Datum predDatum;
+ bool isnull;
+ Datum result;
+
+ predDatum = SysCacheGetAttr(INDEXRELID, indexRelation->rd_indextuple,
+ Anum_pg_index_indpred, &isnull);
+
+ result = DirectFunctionCall2(pg_get_expr,
+ predDatum,
+ ObjectIdGetDatum(relid));
+
+ p_info->indpred_str = text_to_cstring(DatumGetTextP(result));
+ }
+
+ index_close(indexRelation, NoLock);
+
+ return p_info;
}
static void
if (prev_get_relation_info)
(*prev_get_relation_info) (root, relationObjectId, inhparent, rel);
- /* Do nothing if we don't have valid hint in this context. */
- if (!current_hint)
+ /*
+ * Do nothing if we don't have valid hint in this context or current
+ * nesting depth is nesting depth of SPI calls.
+ */
+ if (!current_hint || nested_level > 0)
return;
if (inhparent)
{
/* store does relids of parent table. */
current_hint->parent_relid = rel->relid;
+ current_hint->parent_rel_oid = relationObjectId;
}
else if (current_hint->parent_relid != 0)
{
appinfo->child_relid == rel->relid)
{
if (current_hint->parent_hint)
- delete_indexes(current_hint->parent_hint, rel);
+ delete_indexes(current_hint->parent_hint, rel,
+ relationObjectId);
return;
}
/* This rel is not inherit table. */
current_hint->parent_relid = 0;
+ current_hint->parent_rel_oid = InvalidOid;
current_hint->parent_hint = NULL;
}
}
set_scan_config_options(hint->enforce_mask, current_hint->context);
hint->base.state = HINT_STATE_USED;
+
if (inhparent)
+ {
+ Relation relation;
+ List *indexoidlist;
+ ListCell *l;
+
current_hint->parent_hint = hint;
- delete_indexes(hint, rel);
+ relation = heap_open(relationObjectId, NoLock);
+ indexoidlist = RelationGetIndexList(relation);
+
+ foreach(l, indexoidlist)
+ {
+ Oid indexoid = lfirst_oid(l);
+ char *indexname = get_rel_name(indexoid);
+ bool use_index = false;
+ ListCell *lc;
+ ParentIndexInfo *parent_index_info;
+
+ foreach(lc, hint->indexnames)
+ {
+ if (RelnameCmp(&indexname, &lfirst(lc)) == 0)
+ {
+ use_index = true;
+ break;
+ }
+ }
+ if (!use_index)
+ continue;
+
+ parent_index_info = get_parent_index_info(indexoid,
+ relationObjectId);
+ current_hint->parent_index_infos =
+ lappend(current_hint->parent_index_infos, parent_index_info);
+ }
+ heap_close(relation, NoLock);
+ }
+ else
+ delete_indexes(hint, rel, InvalidOid);
}
/*
return NULL;
}
+static Relids
+OuterInnerJoinCreate(OuterInnerRels *outer_inner, LeadingHint *leading_hint,
+ PlannerInfo *root, List *initial_rels, HintState *hstate, int nbaserel)
+{
+ OuterInnerRels *outer_rels;
+ OuterInnerRels *inner_rels;
+ Relids outer_relids;
+ Relids inner_relids;
+ Relids join_relids;
+ JoinMethodHint *hint;
+
+ if (outer_inner->relation != NULL)
+ {
+ return bms_make_singleton(
+ find_relid_aliasname(root, outer_inner->relation,
+ initial_rels,
+ leading_hint->base.hint_str));
+ }
+
+ outer_rels = lfirst(outer_inner->outer_inner_pair->head);
+ inner_rels = lfirst(outer_inner->outer_inner_pair->tail);
+
+ outer_relids = OuterInnerJoinCreate(outer_rels,
+ leading_hint,
+ root,
+ initial_rels,
+ hstate,
+ nbaserel);
+ inner_relids = OuterInnerJoinCreate(inner_rels,
+ leading_hint,
+ root,
+ initial_rels,
+ hstate,
+ nbaserel);
+
+ join_relids = bms_add_members(outer_relids, inner_relids);
+
+ if (bms_num_members(join_relids) > nbaserel)
+ return join_relids;
+
+ /*
+ * If we don't have join method hint, create new one for the
+ * join combination with all join methods are enabled.
+ */
+ hint = find_join_hint(join_relids);
+ if (hint == NULL)
+ {
+ /*
+ * Here relnames is not set, since Relids bitmap is sufficient to
+ * control paths of this query afterward.
+ */
+ hint = (JoinMethodHint *) JoinMethodHintCreate(
+ leading_hint->base.hint_str,
+ HINT_LEADING,
+ HINT_KEYWORD_LEADING);
+ hint->base.state = HINT_STATE_USED;
+ hint->nrels = bms_num_members(join_relids);
+ hint->enforce_mask = ENABLE_ALL_JOIN;
+ hint->joinrelids = bms_copy(join_relids);
+ hint->inner_nrels = bms_num_members(inner_relids);
+ hint->inner_joinrelids = bms_copy(inner_relids);
+
+ hstate->join_hint_level[hint->nrels] =
+ lappend(hstate->join_hint_level[hint->nrels], hint);
+ }
+ else
+ {
+ hint->inner_nrels = bms_num_members(inner_relids);
+ hint->inner_joinrelids = bms_copy(inner_relids);
+ }
+
+ return join_relids;
+}
+
/*
* Transform join method hint into handy form.
*
* - add join method hints which are necessary to enforce join order
* specified by Leading hint
*/
-static void
+static bool
transform_join_hints(HintState *hstate, PlannerInfo *root, int nbaserel,
List *initial_rels, JoinMethodHint **join_method_hints)
{
int i;
int relid;
- LeadingHint *lhint;
Relids joinrelids;
int njoinrels;
ListCell *l;
+ char *relname;
+ LeadingHint *lhint = NULL;
/*
* Create bitmap of relids from alias names for each join method hint.
relid = 0;
for (j = 0; j < hint->nrels; j++)
{
- char *relname = hint->relnames[j];
+ relname = hint->relnames[j];
relid = find_relid_aliasname(root, relname, initial_rels,
hint->base.hint_str);
/* Do nothing if no Leading hint was supplied. */
if (hstate->num_hints[HINT_TYPE_LEADING] == 0)
- return;
+ return false;
- /* Do nothing if Leading hint is invalid. */
- lhint = hstate->leading_hint;
- if (!hint_state_enabled(lhint))
- return;
+ /*
+ * Decide to use Leading hint。
+ */
+ for (i = 0; i < hstate->num_hints[HINT_TYPE_LEADING]; i++)
+ {
+ LeadingHint *leading_hint = (LeadingHint *)hstate->leading_hint[i];
+ Relids relids;
+
+ if (leading_hint->base.state == HINT_STATE_ERROR)
+ continue;
+
+ relid = 0;
+ relids = NULL;
+
+ foreach(l, leading_hint->relations)
+ {
+ relname = (char *)lfirst(l);;
+
+ relid = find_relid_aliasname(root, relname, initial_rels,
+ leading_hint->base.hint_str);
+ if (relid == -1)
+ leading_hint->base.state = HINT_STATE_ERROR;
+
+ if (relid <= 0)
+ break;
+
+ if (bms_is_member(relid, relids))
+ {
+ hint_ereport(leading_hint->base.hint_str,
+ ("Relation name \"%s\" is duplicated.", relname));
+ leading_hint->base.state = HINT_STATE_ERROR;
+ break;
+ }
+
+ relids = bms_add_member(relids, relid);
+ }
+
+ if (relid <= 0 || leading_hint->base.state == HINT_STATE_ERROR)
+ continue;
+
+ if (lhint != NULL)
+ {
+ hint_ereport(lhint->base.hint_str,
+ ("Conflict %s hint.", HintTypeName[lhint->base.type]));
+ lhint->base.state = HINT_STATE_DUPLICATION;
+ }
+ leading_hint->base.state = HINT_STATE_USED;
+ lhint = leading_hint;
+ }
+
+ /* check to exist Leading hint marked with 'used'. */
+ if (lhint == NULL)
+ return false;
/*
* We need join method hints which fit specified join order in every join
*/
joinrelids = NULL;
njoinrels = 0;
- foreach(l, lhint->relations)
+ if (lhint->outer_inner == NULL)
{
- char *relname = (char *)lfirst(l);
- JoinMethodHint *hint;
-
- /*
- * Find relid of the relation which has given name. If we have the
- * name given in Leading hint multiple times in the join, nothing to
- * do.
- */
- relid = find_relid_aliasname(root, relname, initial_rels,
- hstate->hint_str);
- if (relid == -1)
+ foreach(l, lhint->relations)
{
- bms_free(joinrelids);
- return;
- }
- if (relid == 0)
- continue;
+ JoinMethodHint *hint;
- /* Found relid must not be in joinrelids. */
- if (bms_is_member(relid, joinrelids))
- {
- hint_ereport(lhint->base.hint_str,
- ("Relation name \"%s\" is duplicated.", relname));
- lhint->base.state = HINT_STATE_ERROR;
- bms_free(joinrelids);
- return;
- }
+ relname = (char *)lfirst(l);
- /* Create bitmap of relids for current join level. */
- joinrelids = bms_add_member(joinrelids, relid);
- njoinrels++;
+ /*
+ * Find relid of the relation which has given name. If we have the
+ * name given in Leading hint multiple times in the join, nothing to
+ * do.
+ */
+ relid = find_relid_aliasname(root, relname, initial_rels,
+ hstate->hint_str);
- /* We never have join method hint for single relation. */
- if (njoinrels < 2)
- continue;
+ /* Create bitmap of relids for current join level. */
+ joinrelids = bms_add_member(joinrelids, relid);
+ njoinrels++;
+
+ /* We never have join method hint for single relation. */
+ if (njoinrels < 2)
+ continue;
- /*
- * If we don't have join method hint, create new one for the
- * join combination with all join methods are enabled.
- */
- hint = find_join_hint(joinrelids);
- if (hint == NULL)
- {
/*
- * Here relnames is not set, since Relids bitmap is sufficient to
- * control paths of this query afterward.
+ * If we don't have join method hint, create new one for the
+ * join combination with all join methods are enabled.
*/
- hint = (JoinMethodHint *) JoinMethodHintCreate(lhint->base.hint_str,
- HINT_LEADING);
- hint->base.state = HINT_STATE_USED;
- hint->nrels = njoinrels;
- hint->enforce_mask = ENABLE_ALL_JOIN;
- hint->joinrelids = bms_copy(joinrelids);
- }
+ hint = find_join_hint(joinrelids);
+ if (hint == NULL)
+ {
+ /*
+ * Here relnames is not set, since Relids bitmap is sufficient
+ * to control paths of this query afterward.
+ */
+ hint = (JoinMethodHint *) JoinMethodHintCreate(
+ lhint->base.hint_str,
+ HINT_LEADING,
+ HINT_KEYWORD_LEADING);
+ hint->base.state = HINT_STATE_USED;
+ hint->nrels = njoinrels;
+ hint->enforce_mask = ENABLE_ALL_JOIN;
+ hint->joinrelids = bms_copy(joinrelids);
+ }
- join_method_hints[njoinrels] = hint;
+ join_method_hints[njoinrels] = hint;
- if (njoinrels >= nbaserel)
- break;
- }
+ if (njoinrels >= nbaserel)
+ break;
+ }
+ bms_free(joinrelids);
- bms_free(joinrelids);
+ if (njoinrels < 2)
+ return false;
- if (njoinrels < 2)
- return;
+ /*
+ * Delete all join hints which have different combination from Leading
+ * hint.
+ */
+ for (i = 2; i <= njoinrels; i++)
+ {
+ list_free(hstate->join_hint_level[i]);
- /*
- * Delete all join hints which have different combination from Leading
- * hint.
- */
- for (i = 2; i <= njoinrels; i++)
+ hstate->join_hint_level[i] = lappend(NIL, join_method_hints[i]);
+ }
+ }
+ else
{
- list_free(hstate->join_hint_level[i]);
+ joinrelids = OuterInnerJoinCreate(lhint->outer_inner,
+ lhint,
+ root,
+ initial_rels,
+ hstate,
+ nbaserel);
+
+ njoinrels = bms_num_members(joinrelids);
+ Assert(njoinrels >= 2);
+
+ /*
+ * Delete all join hints which have different combination from Leading
+ * hint.
+ */
+ for (i = 2;i <= njoinrels; i++)
+ {
+ if (hstate->join_hint_level[i] != NIL)
+ {
+ ListCell *prev = NULL;
+ ListCell *next = NULL;
+ for(l = list_head(hstate->join_hint_level[i]); l; l = next)
+ {
+
+ JoinMethodHint *hint = (JoinMethodHint *)lfirst(l);
+
+ next = lnext(l);
+
+ if (hint->inner_nrels == 0 &&
+ !(bms_intersect(hint->joinrelids, joinrelids) == NULL ||
+ bms_equal(bms_union(hint->joinrelids, joinrelids),
+ hint->joinrelids)))
+ {
+ hstate->join_hint_level[i] =
+ list_delete_cell(hstate->join_hint_level[i], l,
+ prev);
+ }
+ else
+ prev = l;
+ }
+ }
+ }
- hstate->join_hint_level[i] = lappend(NIL, join_method_hints[i]);
+ bms_free(joinrelids);
}
if (hint_state_enabled(lhint))
+ {
set_join_config_options(DISABLE_ALL_JOIN, current_hint->context);
-
- lhint->base.state = HINT_STATE_USED;
-
+ return true;
+ }
+ return false;
}
/*
if (!hint)
return pg_hint_plan_make_join_rel(root, rel1, rel2);
- save_nestlevel = NewGUCNestLevel();
+ if (hint->inner_nrels == 0)
+ {
+ save_nestlevel = NewGUCNestLevel();
- set_join_config_options(hint->enforce_mask, current_hint->context);
+ set_join_config_options(hint->enforce_mask, current_hint->context);
- rel = pg_hint_plan_make_join_rel(root, rel1, rel2);
- hint->base.state = HINT_STATE_USED;
+ rel = pg_hint_plan_make_join_rel(root, rel1, rel2);
+ hint->base.state = HINT_STATE_USED;
- /*
- * Restore the GUC variables we set above.
- */
- AtEOXact_GUC(true, save_nestlevel);
+ /*
+ * Restore the GUC variables we set above.
+ */
+ AtEOXact_GUC(true, save_nestlevel);
+ }
+ else
+ rel = pg_hint_plan_make_join_rel(root, rel1, rel2);
return rel;
}
+/*
+ * TODO : comment
+ */
+static void
+add_paths_to_joinrel_wrapper(PlannerInfo *root,
+ RelOptInfo *joinrel,
+ RelOptInfo *outerrel,
+ RelOptInfo *innerrel,
+ JoinType jointype,
+ SpecialJoinInfo *sjinfo,
+ List *restrictlist)
+{
+ ScanMethodHint *scan_hint = NULL;
+ Relids joinrelids;
+ JoinMethodHint *join_hint;
+ int save_nestlevel;
+
+ if ((scan_hint = find_scan_hint(root, innerrel)) != NULL)
+ {
+ set_scan_config_options(scan_hint->enforce_mask, current_hint->context);
+ scan_hint->base.state = HINT_STATE_USED;
+ }
+
+ joinrelids = bms_union(outerrel->relids, innerrel->relids);
+ join_hint = find_join_hint(joinrelids);
+ bms_free(joinrelids);
+
+ if (join_hint && join_hint->inner_nrels != 0)
+ {
+ save_nestlevel = NewGUCNestLevel();
+
+ if (bms_equal(join_hint->inner_joinrelids, innerrel->relids))
+ {
+
+ set_join_config_options(join_hint->enforce_mask,
+ current_hint->context);
+
+ add_paths_to_joinrel(root, joinrel, outerrel, innerrel, jointype,
+ sjinfo, restrictlist);
+ join_hint->base.state = HINT_STATE_USED;
+ }
+ else
+ {
+ set_join_config_options(DISABLE_ALL_JOIN, current_hint->context);
+ add_paths_to_joinrel(root, joinrel, outerrel, innerrel, jointype,
+ sjinfo, restrictlist);
+ }
+
+ /*
+ * Restore the GUC variables we set above.
+ */
+ AtEOXact_GUC(true, save_nestlevel);
+ }
+ else
+ add_paths_to_joinrel(root, joinrel, outerrel, innerrel, jointype,
+ sjinfo, restrictlist);
+
+ if (scan_hint != NULL)
+ set_scan_config_options(current_hint->init_scan_mask,
+ current_hint->context);
+}
+
static int
get_num_baserels(List *initial_rels)
{
pg_hint_plan_join_search(PlannerInfo *root, int levels_needed,
List *initial_rels)
{
- JoinMethodHint **join_method_hints;
- int nbaserel;
- RelOptInfo *rel;
- int i;
+ JoinMethodHint **join_method_hints;
+ int nbaserel;
+ RelOptInfo *rel;
+ int i;
+ bool leading_hint_enable;
/*
* Use standard planner (or geqo planner) if pg_hint_plan is disabled or no
- * valid hint is supplied.
+ * valid hint is supplied or current nesting depth is nesting depth of SPI
+ * calls.
*/
- if (!current_hint)
+ if (!current_hint || nested_level > 0)
{
if (prev_join_search)
return (*prev_join_search) (root, levels_needed, initial_rels);
current_hint->join_hint_level = palloc0(sizeof(List *) * (nbaserel + 1));
join_method_hints = palloc0(sizeof(JoinMethodHint *) * (nbaserel + 1));
- transform_join_hints(current_hint, root, nbaserel, initial_rels,
- join_method_hints);
+ leading_hint_enable = transform_join_hints(current_hint, root, nbaserel,
+ initial_rels, join_method_hints);
rel = pg_hint_plan_standard_join_search(root, levels_needed, initial_rels);
pfree(current_hint->join_hint_level);
pfree(join_method_hints);
- if (current_hint->num_hints[HINT_TYPE_LEADING] > 0 &&
- hint_state_enabled(current_hint->leading_hint))
+ if (leading_hint_enable)
set_join_config_options(current_hint->init_join_mask,
current_hint->context);
}
}
+/*
+ * stmt_beg callback is called when each query in PL/pgSQL function is about
+ * to be executed. At that timing, we save query string in the global variable
+ * plpgsql_query_string to use it in planner hook. It's safe to use one global
+ * variable for the purpose, because its content is only necessary until
+ * planner hook is called for the query, so recursive PL/pgSQL function calls
+ * don't harm this mechanism.
+ */
+static void
+pg_hint_plan_plpgsql_stmt_beg(PLpgSQL_execstate *estate, PLpgSQL_stmt *stmt)
+{
+ if ((enum PLpgSQL_stmt_types) stmt->cmd_type == PLPGSQL_STMT_EXECSQL)
+ {
+ PLpgSQL_expr *expr = ((PLpgSQL_stmt_execsql *) stmt)->sqlstmt;
+ plpgsql_query_string = expr->query;
+ }
+}
+
+/*
+ * stmt_end callback is called then each query in PL/pgSQL function has
+ * finished. At that timing, we clear plpgsql_query_string to tell planner
+ * hook that next call is not for a query written in PL/pgSQL block.
+ */
+static void
+pg_hint_plan_plpgsql_stmt_end(PLpgSQL_execstate *estate, PLpgSQL_stmt *stmt)
+{
+ if ((enum PLpgSQL_stmt_types) stmt->cmd_type == PLPGSQL_STMT_EXECSQL)
+ plpgsql_query_string = NULL;
+}
+
#define standard_join_search pg_hint_plan_standard_join_search
#define join_search_one_level pg_hint_plan_join_search_one_level
#define make_join_rel make_join_rel_wrapper
#undef make_join_rel
#define make_join_rel pg_hint_plan_make_join_rel
-#define add_paths_to_joinrel(root, joinrel, outerrel, innerrel, jointype, sjinfo, restrictlist) \
-do { \
- ScanMethodHint *hint = NULL; \
- if ((hint = find_scan_hint((root), (innerrel))) != NULL) \
- { \
- set_scan_config_options(hint->enforce_mask, current_hint->context); \
- hint->base.state = HINT_STATE_USED; \
- } \
- add_paths_to_joinrel((root), (joinrel), (outerrel), (innerrel), (jointype), (sjinfo), (restrictlist)); \
- if (hint != NULL) \
- set_scan_config_options(current_hint->init_scan_mask, current_hint->context); \
-} while(0)
+#define add_paths_to_joinrel add_paths_to_joinrel_wrapper
#include "make_join_rel.c"
+
+#include "pg_stat_statements.c"
OSDN Git Service
