Improve error message in `sqllogictest` with `<slt:ignore>` marker
- Langage dominant
- Rust
- Étoiles
- 9.3k
- Forks
- 2.4k
- Merge moyen
- 3 j 11 h
- PR mergées (30 j)
- 360
Description
### Is your feature request related to a problem or challenge?
Follow up to https://github.com/apache/datafusion/pull/18857
With `` feature, if the actual and expected is not matched, we got error message like
```
2. query result mismatch:
[SQL] select * from generate_series(3);
[Diff] (-expected|+actual)
-
+ 0
1
- 3
+ 2
3
at /Users/yongting/Code/datafusion/datafusion/sqllogictest/test_files/slt_features.slt:84
```
The first diff is a false positive, and the second one is expected. It would be great to suppress the 1st diff.
Also, the error message rendering and the actual validation for `sqllogictest` seems to live in two different places, it would be better to combine them.
### Describe the solution you'd like
_No response_
### Describe alternatives you've considered
_No response_
### Additional context
_No response_
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Reproduisez l’écart dans datafusion/sqllogictest/test_files/slt_features.slt à la ligne 84, puis suivez les emplacements distincts de validation et de rendu des erreurs de sqllogictest mentionnés dans l’issue. La modification terminée doit supprimer le diff faussement positif causé par tout en signalant toujours l’écart réel, avec une couverture pour cette sortie.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- rust
- Domaine
- testing-qa
- Type d'issue
- Fonctionnalité
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 45/100