Restore current hint state when returned from non-hinted query planning.

If no hint is given for the current level query, pg_hint_plan_planner
calls the next level of planner after erasing the
current_hint_state. But it forgot to restore the state before the
planning of the rest part of the current-level query.  It is broken by
the commit 958c60d but overlooked among vast amount of behavioral
changes..  Get back the behavior by restoring current_hint_state after
returned from the lower level planner with unhinted query.

Issue: https://github.com/ossc-db/pg_hint_plan/issues/30
Reported-by: higuchi-daisuke

Fix Rows hint parsing

This is a long standing bug that Rows hint with no parameter causees a
crash. Fixed the Rows hint parser.

Fix crash bug caused by plancache invalidation

https://github.com/ossc-db/pg_hint_plan/issues/41

After plancache is invalidated then revaliated, get_query_string
accesses query_list of invalid plansource then crash. Ignore invalid
plancache and get the correct hint string and Query node at the next
planning time during revalidation.

On the way fixing this, a bug related to planner reentrance is
fixed. That fix causes behavioral change for nested
planning. Previously outer-level hint (wrongly) overrides inner-level
query but currenlty outer-level hint no longer affects inner-level
query.

Fix  compiling error on Solaris

The 'LDFLAGS+=-Wl,--build-id' option is for rpm building in Linux like
OS.  The SunOS is not native support the option
'LDFLAGS+=-Wl,--build-id' . So, we disable this option in SunOS while
compiling.

Committed a bit tweaked version from the orignal.

Stabilize regression test and catching-up to the core.

Further unstability is found about stats reset. Inserted more 1 second
sleeps to stabilize it. Some planner fix (maybe 72a626e688) seems to
have changed its behvavior. The commit 041ad9a66d changes the result
of A-8-4-10 in ut-A. The commit 88275ac199, which changed the category
of client_min_messages, affected ut-A's result. Follow the changes.

Most of this fix is comes from the pull request
https://github.com/ossc-db/pg_hint_plan/pull/28 by Dagouhan (smoon
<30999182+Dagouhan@users.noreply.github.com>).

Do null-check of error_context_stack before use

The variable is accessed without null checking when planner is called
in the context of pl/pgsql. Fix it. Back-patched up to pg_hint_plan94.

Reporter: egashira-yusuke
Issue: https://github.com/ossc-db/pg_hint_plan/issues/42
Original-Author: sun-wei <30999182+Dagouhan@users.noreply.github.com>
Original-patch: https://github.com/ossc-db/pg_hint_plan/pull/45/commits/8fe092ef73a9af6cfb41e20717d7cb35aa74fce2

fix spelling mistakes in file pg_hint_plan.html

Change version to 1.1.8

Version bumped to 1.1.8. SPEC file is updated.

Fix reverse link of hint_list.html

pg_hint_plan.html and hint_list.html pointed to the Japanese version
of main document. Fixed it.

Fix documentation

Fix wrong or stale descriptions in documentation and fix style.

Do make install for rpmbuild

Use make install to build in prmbuild directory instaed of using
install command to individual file to reduce maintenance labor.

Include all pg_hint_plan--*.sql files in installation

Glob installation/upgrade SQL scripts instead of spcifying
individually to reduce maintenance labor.

Change copyright year to 2019

With some minor fixes in the document.

Call standrad_ProcessUtility when no hook is set

The last commit 6973627 forgot the case of no previous hook function
in the ProcessUtility hook function. Fix it.

Support prepared statements on extended protocol

However pg_hint_plan doesn't fully consider the extended protocol,
commit 64ab47b accidentially broke the case where an analyzed prepared
statement is executed on extended protocol. This patch fixes that only
for the hints-in-comment case. Hint-table still doesn't work in the
case since query-normalization needs Query, which is not available in
planner_hook.

Remove an useless variable

commit 64ab47b made the variable stmt_name useless. Remove it.

Fix for union-on-inheritance case

setup_hint_enforcement may pick up a wrong hint when multiple
subqueries access inheritance tables. This leads to failure to apply
hints on other than the first subquery.

Don't reset plpgsql nest level at every command end

Clean up code for global exit is forgetting about loops that run
hinted commands multiple times. This leads to pg_hint_plan's failure
to affect after the first execution of the inner commands.
The hint below worked only for the first time.

FOR outer IN EXECUTE 'outer-command' LOOP
  FOR inner IN EXECUTE '/*+ hint */ inner-command' LOOP
    <something>
  END LOOP;
END LOOP;

Changed version to 1.1.7

Previously parse_messages and message_level were binded to the same
internal variable. Separate the two GUCs. On the way doing this some
mssages that did not honor the variable were fixed. Default behavior
is not changed but the the two variables works differently from the
previous version, perhaps closer to expected.

Fix debug output level control

Previously parse_messages and message_level were binded to the same
internal variable. Separate the two GUCs. On the way doing this some
mssages that did not honor the variable were fixed. Default behavior
is not changed but the the two variables works differently from the
previous version, perhaps closer to expected.

Change version to 1.1.6.

1.1.5 has a crash bug in Describe message of extended protocol
path. If server receives the message after invalidation of
corresponding query cache, pg_hint_plan crashes.

On the way fixing it, this version took in the following commit of
core.

80e12a6218 Change more places to be less trusting of RestrictInfo.is_pushed_down.

Took in core improvement.

Took in the following commit of core.

80e12a6218 Change more places to be less trusting of RestrictInfo.is_pushed_down.

Fixed typos and removed useless include on the way.

Fix a crash bug in case debug_query_string is NULL

pg_hint_plan believed that debug_query_string cannot be null when
parse_analyze is called, but for example in the case under
exec_describe_statement_message, it is not. We see the query string in
pstate even in the case, so use it instead in the case. Since pstate
is storing the query of the lowermost level, we should use
debug_query_string in other cases.

Added and removed some entries in .gitignore

Removed core.c, that is not appropriate.
Added .deps and RPMS.

Use pg_strcasecmp instead of strcasecmp for the sake of portability

Added a notice in the manual.

Added an item that explains about letter case handling of object names
in hints.

Change version to 1.1.5

1.1.4 has an assertion failure bug raised by DECLARE CURSOR with
enabling table hint. Fixed it.

Fixed a crash bug by DECLARE CURSOR and enable_hint_table = on

The previous version causes assertion failure by DECLARE CURSOR syntax
when table hint is activated. The cause is that the version forgot the
fact that DelcareCursorStmt is in a bit strange shape. Add support of
DECLARE CURSOR and regression test for table hinting.

Change version to 1.1.4

1.1.3 has a crash bug raised by queries in specific shape involving
complex views and/or SQL functions.

It is fixed in this version.

Fix a bug of target query retrieval

The previous patch leaves a bug that get_query_string returns a Query
that JumbleQuery doesn't accept. This fixes that bug.

Fix a crash bug on complex views when enable_hint_table is on

The Query that planner receives sometimes irrelevant to
debug_query_string. If enable_hint_table is on, pg_hint_plan_planner
normalizes debug_query_string using query-jumble information created
from the irrelevant Query the can lead to crash. To avoid this
situation, retrieve hints in post_parse_analyze_hook, where
corresponding pairs of a query string and a parsed Query.

Add files needed to do installcheck into tar ball.

sql/maskout.sh and data/data.csv are necessary to perform installcheck
but ignored in making tarball. COPYRIHT* files are also included by
this commit. Although version tags won't be moved since it brings out
no functional changes, the tarballs already released will be replaced
with new one applied this fix later.

Pointed by Tomokuni Fujita and fixed by me.

Add RPM spec file forgotten in the previous commit.

Support PostgreSQL 9.5.0.

This branch PG95 has been a bit too early so it needs an additional
merge from master branch to complete to support PostgreSQL 9.5.

Prepare for 1.1.3/PG95.

The pg_hint_plan version of master branch is left as 1.1.2. This
commit corrects the pg_hint_plan version, and edits RPM building
environment for PostgreSQL 9.5.

Edit copyright years.

Change for new syntax items in 9.5.

Some syntactical elements are added in 9.5. This change reflects it.

Addition to that, deleted unused parts in pg_stat_statements, which
are previously invalidated with #ifdef directives.

Follow a change of error message.

Context message of PL/pgSQL is changed so this commit refrects it to
exted/pg_hint_plan.out.

Followed the changes of 9.5.0 release.

There was some changes in 9.5.0 release affect
pg_hint_plan. set_append_rel_pathlist() no longer sets cheapest path
and it became a business of the caller. So rebuild_scan_path() does
so. core.c gets changed from changing lateral join infrastracture
(acfcd45cacb6df23edba4cb3753a2be594238a99) and a change related to custom path(c2ea2285e978d9289084846a3343cef7d261d880).

Made compatible with 9.5a1

Made make_join_rel.c to catch up REL9_5_ALPHA2.  Changed regression
tests following some differences of PostgreSQL's behavior in such like
collapsing of simple VALUES clause, rows estimation and explain
representation of join filter.

Made compatible with 9.5a1

Make make_join_rel.c to catch up to REL9_5_ALPHA2.  Change regtests to
reflect some differences of PostgreSQL's behavior in such like
collapsing of simple VALUES clause, rows estimation and explain
representation of join filter.

Fixed a bug related to SQL statements in PL/pgSQL functions.

At least since 9.1, true query strings of SQL statements executed in
PL/pgSQL functions were found that were not obtained correctly by the
previous implement, it is because that PLpgSQL_stmt variable did not
have proper query string to be read for hints. Instead, it is changed
to read them from the top of error_context_stack in
pg_hint_plan_planner(). This change made a slight difference in
behavior which doesn't affect its work so a part of the regtest was
also changed. And added some regression tests for it.

Catch up the changes of pg_stat_statements.c

The change has nothing to do with pg_hint_plan but it's necessary to
catch it up in order to keep the difference minimal for the ease of
maintenance.

Set additional parameter of set_config_option().

9.5dev added new parameter to set_config_option(). This paramter
should be false in set_config_option_wrapper.

Fix regression test according to the changes of PostgreSQL.

PostgreSQL became to give WARNING for usage of hash indexes. This
change is to follow it.

Prepare for 1.1.2

Remove unnecessary tracker code from documentations.

Added English documentation.

Some fix and addition to the japaneses document.

It lacked the description about joni direction enforcement so added
it. Addition to it, added html language specification and updated
target PostgreSQL versions and OS version.

Fix regression test according to the changes of PostgreSQL.

Row countings of EXPLAIN results seems to be changed. This follows the
change.

Removed a stale comment.

Fix to remove plpgsql query string on non-local exit.

Pl/pgsql query string stored internally squatted after an execution
error then pg_hint_plan afterward read it for hints and ignores really
given ones. This commit makes it erased at the end of transaction.

Fix about unexpectedly living plpgsql query string.

pg_hint_plan forgot to erase plpgsql query strings when it is NOT a
static statement, so it continued to read hints from the remenbered
wrong query string after using dynamic execution of a statement. This
commit makes it to be erased for any types of pl/pgsql statement.

Enhance debug output.

Added new debug output levels detailed and verbose. These let you see
more verbose information about what pg_hint_plan does internally.

Revert "Add new join method hint "NestLoop_NM" which means force nested loop"

This reverts commit 7f6e3ce1326751c36676653ef1e81893296860b6.

Add new join method hint "NestLoop_NM" which means force nested loop
but inhibit materialize of the inner relation.

Fix a bug that index restriction by name doesn't work for UPDATEs on
inheritance parent.

Inheritance planner doesn't claim for the inheritance parent relation,
so pg_hint_plan_get_relation_info cannot have the chance to prepare
the index spec information for chlid relations. This change make the
function to try to find parent relation even if it is called for the
children with inhparent == false from the first.

Remove unused struct member.

RPM automation

Change verision to 1.1.1. Change target version to 9.4.

Make codes copied from PostgreSQL catch up to 9.4

Interim support for 9.4beta2. pg_stat_statements.c is replaced with the latest version. Regtests are rearranged and stabilized.

Merge branch 'master' into PG93

リグレッションテストを出力メッセージの変更に対応させた。

hint_reportの出力メッセージを大文字始まりで統一した。

コメントやメッセージを修正した。

コピーライトを更新した

Rowsで外部ファイルを扱う試験を環境に依存しないようにした。

外部ファイルのアドレスを特定の文字列に置き換えることで、環境に依存
しないようにした。

Rows関連試験の予測結果を、全試験を流した場合のものに変更した。

Rowsヒント句のメッセージ出力(R-3-6-*)の試験を追加した。

Rowsヒント句の同じテーブル組合せ(R-3-5-*)の試験を追加した。

Rowsヒント句の継承テーブル関連(R-3-4-*)の試験を追加した。

Rowsヒント句の比率指定(R-3-3-*)の試験を追加した。

Rowsヒント句の増減値指定(R-3-2-*)の試験を追加した。

Rowsヒント句の絶対値指定(R-3-1-*)の試験を追加した。

bug fix #3

R-2-4-*の試験とR-2-5-*の試験を追加した。

Rowsヒント句のRULEまたはVIEWの試験(R-2-3-*)を追加した。

バグ修正にそって試験および予想結果を修正した。

bug fix #2

件数補正のときに、全ての結合パスにおいて部分集合となってしまい、他
のRowsヒント句適用後に見積もり件数を再計算してしまった。。

理由
  Rowsヒント句に存在しないテーブル組合せを指定すると、pg_hint_plan
  内部で空のテーブル集合として扱っていたため。

Rowsヒント句の件数の書式に関する記述を事実から要請に変更した。

試験R-1-6-7の予測結果を正しい形に修正した。

1ブロックあたりのテーブル数(R-2-2-*)の試験を追加した。

ただし、不合格

複数の問合せブロックに関する試験(R-2-1-*)を追加した。

ただし、不合格

core.cの各関数をPG93向けに更新した。

マニュアルにRowsヒント句の仕様の一部と具体例を挿入した。

記入内容
・Rowsヒント句の補正の件数や倍率をstrtodで補正していること。
理由
・16進数でも指定可能なため。

Rowsヒント句の引数の条件(R-1-*)の試験を追加した。

機能試験表との試験番号のずれをなおした。

スキャン方式試験の予測結果がSQLの変更に沿っていなかったため修正した

スキャン方式試験のコメントが一部不適切だったため修正した。

スキャン方式試験の番号のずれを修正した。

Fix test numbers.

PG9.3でのコスト値の変動を反映。

PG92ブランチで追加したコメントに追従。

Rowsヒントの追加に伴いリグレッションテスト項目の予想結果を追加。

9.2版からのコピーなので、細かいコスト値は正確ではない。

PG92との差分から生成したパッチを適用。

ただし、PG9.3対応として加えた修正はそのまま残した。

Add *.o to ignore file list.

specファイル中の依存ファイル名を修正した。

RPM版インストール時の依存ファイル名に誤りがあったため

Copyrightの未更新部分を更新した。

コピーライトと一部の流用関数に更新漏れがあったため、修正した。