apache / apache/datafusion

Improve error message in `sqllogictest` with `<slt:ignore>` marker

Ouverte
#18,878 4 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
enhancement
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

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.