チケット #36273

登録: 2016-04-22 14:15

最終更新: 2016-04-22 14:16

regression test using psql and grep

報告者:fujii_masao担当者:(未割り当て)
優先度:5 - 中マイルストーン:(未割り当て)
チケットの種類:パッチ重要度:5 - 中
コンポーネント:(未割り当て)状況:オープン
解決法なし

チケットの詳細

Hi,

\! psql contrib_regression -c "/*+IndexScan(ti1 ti1_pred)*/ EXPLAIN (COSTS true) SELECT * FROM s1.ti1 WHERE c1 = 100" | grep -v "Planning time:"

I found that the regression test (e.g., ut-S.sql) tries to filter out the line containing "Planning time:" from the result of EXPLAIN command, in several places. But ISTM that this filtering is NOT necessary because EXPLAIN basically doesn't output that line (but EXPLAIN ANALYZE does, though).

Therefore, shouldn't we remove psql and grep parts from the tests like the above and just execute EXPLAIN command?

Regards,

添付ファイル

添付ファイルリスト添付ファイルはありません
新規添付ファイル追加
添付ファイルの追加添付ファイルの追加にはログインが必要です

チケットの履歴 - 3 件中 3 件表示

2016-04-22 14:15 更新者: fujii_masao

  • 新しいチケット "regression test using psql and grep" が作成されました

2016-04-22 14:16 更新者: fujii_masao

  • チケットの種類機能リクエスト から パッチ に更新されました

2016-04-26 18:13 更新者: horiguti

コメント

You're right. This is added in the commit 7cd601eb6e8d0dfa9ce9757626ac2c279698a4c7, where it supported 9.4 beta 2. Since that version of PostgreSQL emitted the "Planning time" for (COSTS true), the regression was changed so but it had been changed to the current shape as of 9.4.0 release. As the result, no release version of PostgreSQL emits "Planning time" for (COSTS true) so the greps should be removed.


追記/更新 #36273 (regression test using psql and grep)

ログインしていません。ログインしていない状態では、コメントに記載者の記録が残りません。 » ログインする