pingcap/tidb

some explain test case should ignore plan-id in the result file

Open

#60,940 opened on 2025年4月29日

GitHub で見る
 (4 comments) (0 reactions) (1 assignee)Go (40,090 stars) (6,186 forks)batch import
good first issuesig/plannertype/enhancement

説明

Enhancement

Some explain-test cases will record the plan ID in the result file, which will be a burden when some optimization flow is changed, since plan ID allocation is sequential according to the allocation order or space.

コントリビューターガイド

技術スタック
gosql
領域
databasetesting
Issue 種別
test
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
blocked
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
Go programmingTiDB test framework
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
30
調査方針
Look at the explain test cases in the TiDB repository, likely in the `tests/integration tests` or similar directories. The plan id appears in expected output files. The goal is to ignore the specific plan id field in comparisons, possibly by using a placeholder or by modifying the test framework to skip that field. Check existing test patterns that handle non deterministic output. The fix likely involves updating the test case files or the test runner to strip out plan id values before comparison.
some explain test case should ignore plan-id in the result file · pingcap/tidb#60940 | Good First Issue