終了条件の異なる処理を分離した。

以下のような実行回数が異なる処理を１つのforループの中でまとめて
実行し、必ずbreakでループを終了していたので、処理ごとにループ分
けることでわかりやすくした。
・各ヒントグループのカウント処理
・ヒントの重複チェック処理

全てのヒント方式に共通するパース処理を1つの関数にまとめた。

エラーメッセージのクリーンアップに伴って使用しなくなった変数を削除した。

パース処理時にワードを切り出す関数parse_quote_valueの引数type_value
を削除した。

PG本体に合わせてエラーメッセージを修正した。

elogのメッセージの先頭を小文字にした。
パース処理するときの文字列の字数制限に関するメッセージをPG本体に合
わせた。

使用例セクションを追加しヒントグループごとにSQL文を記述

レビュー管理表No.9

章構成をpg_dbms_statsに合わせた

レビュー管理表No.7

ECPGの動的SQLの記述をSQLDAを使う場合と区別できる記述にした。

レビュー管理表No.3

動作環境に9.2を追加

レビュー管理表No.8

制約事項が多くなったのでグループに分割

併せて用語の統一も行った。
- VIEW、RULE→ビュー、ルール
- エイリアス→別名
レビュー管理表No.6

<a>タグ内のパラメータ名記述忘れを修正

他の箇所と合わせて、リンクではなく通常の文字列とした。
また、message_levelという古いパラメータ名が残っていたので併せて修正した。
レビュー管理表No.5

「クエリに明記されていないテーブルへのヒント適用」に例を追加

文章だけでは分かりづらい内容なので、具体的なクエリを挙げて説明した。
レビュー管理表No.4

文章とEXPALIN例でヒント種別か異なっていたので修正

レビュー管理表No.2

表記ゆれを統一(ダブルクォート→二重引用符)

レビュー管理表No.1

core-9.2.c に余分な改行や改行不足を解消した。

流用もとのファイルと当該ファイルでコピーライトが異なっていたため、一致させた。

Merge branch 'master' into translate_comments

実行していたクエリが試験の目的に沿っていなかったため、修正した。

外部テーブルをヒント対象にしたときの各ヒント方式の試験を追加した。

既存の試験ファイル(fdw.source)を機能試験用のファイルに変更し、外部テ
ーブルを対象とする各ヒントの機能試験をこのファイルにまとめた。

リグレッションテストのリファクタリングを実施した。

- 試験番号のずれを修正した。
- 余分なクエリを削除した。
- 試験の目的に合わせてクエリを修正した。

CREATE TABLE AS EXECUTE の試験を【機能に依存しない】試験ファイル内に移動した。

IndexOnlyScanヒントの試験を【スキャン方式】試験ファイル内に移動した。

プランナを再帰的に呼び出す場合の試験を【機能に依存しない】試験のファイル内に移動した。

プランナを再帰的に呼び出す場合の試験(A-13-*-*)を追加試験として別フ
ァイルに分けていたが、試験の整理にあたり、機能試験として同じファイ
ルにまとめた。

PREPARE/EXECUTEの試験で、ジェネリックプランを使うように修正した。

Translate comments into English.

リグレッションテストをDBクラスタのエンコードの影響を受けなくした。

Makefileをリファクタリングした

パラメータ名変更に伴いリグレッションテストを修正した。

ドキュメントと同様に、ソースコードのパラメータ名を変更した。

パラメータ名をよりわかりやすいものに変更した。

クエリに明記されていないテーブルへのヒント適用についてマニュアルに追記した。

Merge branch 'cvc_fix' of git://moomin/pg_hint_plan into IndexOnly

IndexOnlyScanヒント用のテストを実装した。

用語を統一(GUCパラメータ)

9.2でのcustom_variable_classesの廃止に対応。

postgresql.confでpg_hint_plan独自パラメータを設定する際に、9.1でのみ
custom_variable_classes設定が必要であることを記述した。

CREATE TABLE AS EXECUTE対応のために、ソースコードと試験を修正した。

CREATE TABLE AS EXECUTE対応の準備として動作確認用の試験を追加した。

IndexOnlyScanヒント用のテストを途中まで実装した。

クエリから考え直す項目がある。
その考え直す項目はexpected/indexonly-9.2.outの出力結果を空行にしている。

IndexOnlyScanヒントのコードをScanヒントのグループの最後に移動した。

[No]IndexOnlyScanヒントの試験パターンを追加し、目的コメントを加えた。

IndexOnlyScanヒントの制限の説明をより誤解されにくい形に修正した。

2つのヒントの説明をより誤解されにくい形に修正した。

NoIndexScanヒントの制限の説明で冗長な部分を削除した。

PostgreSQLのバージョン表記で余分な単語を省いた。

IndexOnlyScanヒントの制限の説明文を読みやすい形に修正した。

[No]IndexOnlyScan対応によって生じた注意書きをマニュアルに追記した。

[No]IndexOnlyScanの説明をマニュアルに追記した。

[No]IndexOnlyScanヒントの試験を追加した。

[No]IndexOnlyScanヒントを追加した。

ビルドなどでできる一時ファイルをgitで無視するようにした。

PREPAREの試験を削除した。

PG9.1とPG9.2でそれぞれのPREPAREの挙動を確認するために加えられていた
ので削除した。

初期のリグレッションをマルチバージョンに対応させた。

スキャン方式の試験(ut-S)をマルチバージョンに対応させた。

結合方式の試験(ut-J)をマルチバージョンに対応させた。

結合順序の試験(ut-L)をマルチバージョンに対応させた。

プランナ再帰呼び出しの試験(ut-A2)をマルチバージョンに対応させた。

GUCパラメータの試験(ut-G)をマルチバージョンに対応させた。

機能に依存しない試験(ut-A)をマルチバージョンに対応させた。

リグレッションテストのinitテストをマルチバージョンに対応させた。

リグレッションテストのファイル構成をマルチバージョンに対応可能にした。

PG9.2でのリグレッションテストはまだ通らない。

PG9.2版 RPM 作成用の SPECファイルを作成した。

PG9.1、PG9.2の両方でビルド可能にした。

マニュアルの注意事項から修正済みの内容を削除した。

ヒント使用状況ごとのデバッグログ出力確認試験(試験項目A-13-4-*)を追加した。

再帰呼び出しに関するデバッグログ出力回数(項目A-13-3-*)の試験を追加した。

ヒント使用の確認試験(追加試験項目A-13-2-*)を追加した。

プランナを再帰的に呼び出す場合の試験用ファイルを作成した

Refactor parse_head_comment; simplified code.

Enhance comments.

Move name constatnts near by enum.

This would improve maintenancebility.

Refactor function name.

Don't use the term AllHint.

Also refactored interface of HintCmp so that it doesn't require boolean
parameter "order".

ヒント情報の構造体名を実態を表すものに変更した。

併せて、その構造体型の変数名/引数名も変更した。

pop_hint の説明のコメントを記述した。

PGのコーディング規約に従ってコードを整理した。

PlanHint構造で使用する変数名をわかりやすい名前で統一した。

現在のヒントを指すポインタをスタックからではなく直接得るようにした。

push_hint の説明のコメントを記述した。

また、ヒント用スタックの実装にリストを使用しているが、ヒント用スタ
ックの上下とリストのデータ順の紐付けについて記述した。

PGのコーディング規約に従ってコードを整理した。

ヒント用スタック構造の関数名をわかりやすい名前に変更した。

2行以上連続した空行を、空行1行に修正した。

pg_hint_plan_plannerのクリーンアップ処理を連続で実行するように修正

併せてコメントを追加した。

pop_stack内で中途半端な状態になる期間を短縮した。

無駄なヒントパースの抑止

pg_hint_plan が無効の場合も常にヒント句をパースしてヒント情報を生成
していたので、pg_hint_plan が有効な場合にのみヒント句をパースするよ
うに修正した。

ヒント用スタックが無効になる処理フローを有効となるように修正した。

pg_hint_plan.parse_messagesをerrorに設定した状態でヒントがエラーと
なると、ヒント用スタックのメモリが開放されて無効のままだった。この
状態でヒント句付きクエリを実行すると、無効状態のヒント用スタックを
使おうとしてエラーとなり、サーバが落ちてしまうため、ヒントがエラー
となってもヒント用スタックが有効となるように修正した。

ヒント保存のスタックについてコメントを追加し、余分なものを削除した。

ヒントのメモリの開放を、ヒントをポップする処理の中に含めた。

ヒント保存用に使っていた構造をPG本体のリスト構造に修正した。

現在のヒントを保存する変数の名前を適切なものに変更した。

関数呼び出しをif文の条件部から外し、処理内容をわかりやすくした。

pg_dbms_statsとの同時実行バグに関する修正案の仮実装

ヒント情報の保存の構造をスタック構造にした。

メッセージスタイル修正に伴い、予想結果を修正。

メッセージスタイルの修正漏れを修正。

メッセージスタイルを統一した。

fix bug operation test No.10

planner_hookが再帰的に呼ばれる場合に、プランが部分的にのみ制御
されてしまう問題について、使用上の注意と制約としてドキュメントに
追加した。

Add spec file to support RPM package distribution.

fix bug operation test No.9

PG_CATCH()を抜ける直前のErrorContextにスイッチする処理を削除した。
※PG_RE_THROW()しない場合は、ErrorContextに戻さなくても良い。

fix bug operation test No.8

識別子を切り詰める関数を、本体の関数をcopy & pasteしていたが、直接
関数を呼び出すように修整した。
また、メッセージの出力方針が他と異なるため、ユーザーズマニュアルに
注意として記述した。

fix bug operation test No.5,6,7

pg_hint_plan独自のGUCパラメータをSetヒントに指定した場合の挙動が
ユーザの想定する挙動でないと考えられるため、ユーザーズマニュアルに
制限事項として記述した。

fix bug operation test No.4

geqoアルゴリズムが使用される場合は、結合順序のヒントと結合方式のヒント
が使用されることをドキュメントに記述した。

fix bug No.3.

NAMEDATALENバイト以上のGUCパラメータをヒントに指定された場合も、
NAMEDATALEN - 1バイトに切り詰める対象に追加した。