* 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"
{
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_NOTIDSCAN,
#if PG_VERSION_NUM >= 90200
HINT_KEYWORD_INDEXONLYSCAN,
+ HINT_KEYWORD_INDEXONLYSCANREGEXP,
HINT_KEYWORD_NOINDEXONLYSCAN,
#endif
HINT_KEYWORD_NESTLOOP,
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);
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
{
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 */
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,
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,
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,
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 dump_quote_value_quote_char(StringInfo buf, const char *value,
- char quote_char);
-static const char *parse_quote_value_term_char(const char *str, char **word,
- bool truncate, char term_char);
+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,
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 const HintParser parsers[] = {
{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_NOTIDSCAN, ScanMethodHintCreate, HINT_KEYWORD_NOTIDSCAN},
#if PG_VERSION_NUM >= 90200
{HINT_INDEXONLYSCAN, ScanMethodHintCreate, HINT_KEYWORD_INDEXONLYSCAN},
+ {HINT_INDEXONLYSCANREGEXP, ScanMethodHintCreate,
+ HINT_KEYWORD_INDEXONLYSCANREGEXP},
{HINT_NOINDEXONLYSCAN, ScanMethodHintCreate, HINT_KEYWORD_NOINDEXONLYSCAN},
#endif
{HINT_NESTLOOP, JoinMethodHintCreate, HINT_KEYWORD_NESTLOOP},
};
/*
+ * 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;
}
/*
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;
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->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;
return;
list_free_deep(hint->relations);
- free(hint->outer_inner);
+ if (hint->outer_inner)
+ pfree(hint->outer_inner);
pfree(hint);
}
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
- */
-
-/*
- * Quote value as needed and dump it to buf.
- */
-static void
-dump_quote_value(StringInfo buf, const char *value)
-{
- dump_quote_value_quote_char(buf, value, '"');
-}
-
-/*
- * We specified a character to quote for by addition in quote_char.
- * If there is not a character to quote, we specified '"'.
+ * Copy given value into buf, with quoting with '"' if necessary.
*/
static void
-dump_quote_value_quote_char(StringInfo buf, const char *value, char quote_char)
+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 == '"' || *str == quote_char)
+ 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
-OuterInnerDump(OuterInnerRels *outer_inner, StringInfo buf)
+OuterInnerDesc(OuterInnerRels *outer_inner, StringInfo buf)
{
if (outer_inner->relation == NULL)
{
else
appendStringInfoCharMacro(buf, ' ');
- OuterInnerDump(lfirst(l), buf);
+ OuterInnerDesc(lfirst(l), buf);
}
appendStringInfoCharMacro(buf, ')');
}
else
- dump_quote_value_quote_char(buf, outer_inner->relation, '(');
+ quote_value(buf, outer_inner->relation);
}
static void
-LeadingHintDump(LeadingHint *hint, StringInfo buf)
+LeadingHintDesc(LeadingHint *hint, StringInfo buf)
{
- bool is_first;
- ListCell *l;
-
appendStringInfo(buf, "%s(", HINT_LEADING);
- is_first = true;
if (hint->outer_inner == NULL)
{
+ ListCell *l;
+ bool is_first;
+
+ is_first = true;
+
foreach(l, hint->relations)
{
if (is_first)
else
appendStringInfoCharMacro(buf, ' ');
- dump_quote_value(buf, (char *) lfirst(l));
+ quote_value(buf, (char *) lfirst(l));
}
}
else
- OuterInnerDump(hint->outer_inner, buf);
+ 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)
-{
- return parse_quote_value_term_char(str, word, truncate, '\0');
-}
-
-/*
- * In term_char, We specified the special character which a terminal of a word.
- * When we do not have a special character, We specified '\0'.
- */
-static const char *
-parse_quote_value_term_char(const char *str, char **word, bool truncate,
- char term_char)
+parse_quoted_value(const char *str, char **word, bool truncate)
{
StringInfoData buf;
bool in_quote;
break;
}
}
- else if (isspace(*str) || *str == ')' || *str == '"' || *str == '\0' ||
- *str == term_char)
+ else if (isspace(*str) || *str == '(' || *str == ')' || *str == '"' ||
+ *str == '\0')
break;
appendStringInfoCharMacro(&buf, *str++);
}
static OuterInnerRels *
-OuterInnerRelsCreate(OuterInnerRels *outer_inner, char *name, List *outer_inner_list)
+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;
static const char *
parse_parentheses_Leading_in(const char *str, OuterInnerRels **outer_inner)
{
- char *name;
- bool truncate = true;
+ List *outer_inner_pair = NIL;
- OuterInnerRels *outer_inner_rels;
- outer_inner_rels = NULL;
+ 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++;
- outer_inner_rels = OuterInnerRelsCreate(outer_inner_rels,
- NULL,
- NIL);
str = parse_parentheses_Leading_in(str, &outer_inner_rels);
+ if (str == NULL)
+ break;
}
- else if ((str = parse_quote_value_term_char(str, &name, truncate, '(')) == NULL)
+ else
{
- list_free((*outer_inner)->outer_inner_pair);
- return NULL;
+ char *name;
+
+ if ((str = parse_quoted_value(str, &name, true)) == NULL)
+ break;
+ else
+ outer_inner_rels = OuterInnerRelsCreate(name, NIL);
}
- else
- outer_inner_rels = OuterInnerRelsCreate(outer_inner_rels, name, NIL);
- (*outer_inner)->outer_inner_pair = lappend(
- (*outer_inner)->outer_inner_pair,
- outer_inner_rels);
+ outer_inner_pair = lappend(outer_inner_pair, outer_inner_rels);
skip_space(str);
}
- 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;
}
skip_space(str);
if (*str =='(')
{
- str++;
-
- *outer_inner = OuterInnerRelsCreate(*outer_inner, NULL, NIL);
- str = parse_parentheses_Leading_in(str, outer_inner);
+ 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_quote_value(str, &name, truncate)) == NULL)
+ if ((str = parse_quoted_value(str, &name, truncate)) == NULL)
{
list_free(*name_list);
return NULL;
}
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;
}
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];
/*
- * TODO : Leadingヒントの重複チェックは、transform_join_hints関数の実行
- * 中に実施する。
- */
- if (i >= hstate->num_hints[HINT_TYPE_SCAN_METHOD] +
- hstate->num_hints[HINT_TYPE_JOIN_METHOD] &&
- i < hstate->num_hints[HINT_TYPE_SCAN_METHOD] +
- hstate->num_hints[HINT_TYPE_JOIN_METHOD] +
- hstate->num_hints[HINT_TYPE_LEADING])
+ * Leading hint is marked as 'duplicated' in transform_join_hints.
+ */
+ if (cur_hint->type == HINT_TYPE_LEADING &&
+ next_hint->type == HINT_TYPE_LEADING)
continue;
/*
* 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->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;
}
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 (hint_keyword != HINT_KEYWORD_INDEXSCAN &&
+ if (length != 1 &&
+ hint_keyword != HINT_KEYWORD_INDEXSCAN &&
+ hint_keyword != HINT_KEYWORD_INDEXSCANREGEXP &&
#if PG_VERSION_NUM >= 90200
hint_keyword != HINT_KEYWORD_INDEXONLYSCAN &&
+ hint_keyword != HINT_KEYWORD_INDEXONLYSCANREGEXP &&
#endif
hint_keyword != HINT_KEYWORD_BITMAPSCAN &&
- length != 1)
+ hint_keyword != HINT_KEYWORD_BITMAPSCANREGEXP)
{
hint_ereport(str,
("%s hint accepts only one relation.",
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_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;
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);
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;
OuterInnerRels *outer_inner = NULL;
- if ((str =
- parse_parentheses_Leading(str, &name_list, &outer_inner)) == 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 && hint->outer_inner == NULL)
+ if ( hint->outer_inner == NULL && list_length(hint->relations) < 2)
{
hint_ereport(hint->base.hint_str,
("%s hint requires at least two relations.",
{
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, Oid relationObjectId)
{
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, ' ');
- dump_quote_value(&buf, indexname);
+ quote_value(&buf, indexname);
}
break;
relname = hint->relname;
initStringInfo(&rel_buf);
- dump_quote_value(&rel_buf, relname);
+ quote_value(&rel_buf, relname);
ereport(LOG,
(errmsg("available indexes for %s(%s):%s",
}
}
+/*
+ * 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
pg_hint_plan_get_relation_info(PlannerInfo *root, Oid relationObjectId,
bool inhparent, RelOptInfo *rel)
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)
{
/* 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;
+
+ 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 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 Relids
OuterInnerJoinCreate(OuterInnerRels *outer_inner, LeadingHint *leading_hint,
- PlannerInfo *root, List *initial_rels, HintState *hstate, int *njoinrels,
- int nbaserel)
+ PlannerInfo *root, List *initial_rels, HintState *hstate, int nbaserel)
{
OuterInnerRels *outer_rels;
OuterInnerRels *inner_rels;
if (outer_inner->relation != NULL)
{
- (*njoinrels)++;
return bms_make_singleton(
find_relid_aliasname(root, outer_inner->relation,
initial_rels,
root,
initial_rels,
hstate,
- njoinrels,
nbaserel);
inner_relids = OuterInnerJoinCreate(inner_rels,
leading_hint,
root,
initial_rels,
hstate,
- njoinrels,
nbaserel);
join_relids = bms_add_members(outer_relids, inner_relids);
- if (bms_num_members(join_relids) > nbaserel || *njoinrels > nbaserel)
+ if (bms_num_members(join_relids) > nbaserel)
return join_relids;
/*
* - 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,
- LeadingHint *lhint)
+ List *initial_rels, JoinMethodHint **join_method_hints)
{
int i;
int relid;
int njoinrels;
ListCell *l;
char *relname;
- bool exist_effective_leading;
+ LeadingHint *lhint = NULL;
/*
* Create bitmap of relids from alias names for each join method hint.
/* Do nothing if no Leading hint was supplied. */
if (hstate->num_hints[HINT_TYPE_LEADING] == 0)
- return;
+ return false;
/*
* Decide to use Leading hint。
*/
- exist_effective_leading = false;
- for (i = hstate->num_hints[HINT_TYPE_LEADING] - 1; i >= 0; i--) {
- LeadingHint *leading_hint = (LeadingHint *)hstate->leading_hint[i];
-
- List *relname_list;
- Relids relids;
+ 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;
- if (leading_hint->outer_inner != NULL)
- relname_list = OuterInnerList(leading_hint->outer_inner);
- else
- relname_list = leading_hint->relations;
-
relid = 0;
- relids = 0;
+ relids = NULL;
- foreach(l, relname_list)
+ foreach(l, leading_hint->relations)
{
relname = (char *)lfirst(l);;
if (relid <= 0 || leading_hint->base.state == HINT_STATE_ERROR)
continue;
- if (exist_effective_leading)
- {
- hint_ereport(leading_hint->base.hint_str,
- ("Conflict %s hint.", HintTypeName[leading_hint->base.type]));
- leading_hint->base.state = HINT_STATE_DUPLICATION;
- }
- else
+ if (lhint != NULL)
{
- leading_hint->base.state = HINT_STATE_USED;
- exist_effective_leading = true;
- lhint = leading_hint;
+ 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;
}
- if (exist_effective_leading == false)
- return;
+
+ /* 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
bms_free(joinrelids);
if (njoinrels < 2)
- return;
+ return false;
/*
* Delete all join hints which have different combination from Leading
root,
initial_rels,
hstate,
- &njoinrels,
nbaserel);
+ njoinrels = bms_num_members(joinrelids);
+ Assert(njoinrels >= 2);
+
/*
* Delete all join hints which have different combination from Leading
* hint.
*/
- for (i = 2;i <= nbaserel; i++)
+ for (i = 2;i <= njoinrels; i++)
{
if (hstate->join_hint_level[i] != NIL)
{
}
}
- bms_free(joinrelids);
-
- if (njoinrels < 2)
- return;
+ bms_free(joinrelids);
}
if (hint_state_enabled(lhint))
+ {
set_join_config_options(DISABLE_ALL_JOIN, current_hint->context);
+ return true;
+ }
+ return false;
}
/*
List *initial_rels)
{
JoinMethodHint **join_method_hints;
- LeadingHint *leading_hint = NULL;
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);
+ 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 &&
- leading_hint != NULL &&
- hint_state_enabled(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
#define make_join_rel pg_hint_plan_make_join_rel
#define add_paths_to_joinrel add_paths_to_joinrel_wrapper
#include "make_join_rel.c"
+
+#include "pg_stat_statements.c"