pingcap/tidb

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

Open

#60 940 ouverte le 29 avr. 2025

Voir sur GitHub
 (4 commentaires) (0 réactions) (1 assigné)Go (40 090 stars) (6 186 forks)batch import
good first issuesig/plannertype/enhancement

Description

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.

Guide contributeur

Stack technique
gosql
Domaine
databasetesting
Type d'issue
test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
blocked
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Go programmingTiDB test framework
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30
Direction de recherche
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.