apache / apache/datafusion

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

Offen
#18,878 4 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
enhancement
Vorherrschende Sprache
Rust
Sterne
9.3k
Forks
2.4k
Ø Merge
3 T. 11 Std.
Gemergte PRs (30 T.)
362

Beschreibung

### 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_

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Reproduziere die Abweichung aus datafusion/sqllogictest/test_files/slt_features.slt in Zeile 84 und verfolge anschließend die im Issue genannten separaten Stellen für die sqllogictest-Validierung und die Fehlerdarstellung. Die abgeschlossene Änderung sollte den durch verursachten falsch-positiven Diff unterdrücken und gleichzeitig weiterhin die tatsächliche Abweichung melden, mit Coverage für diese Ausgabe.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
rust
Bereich
testing-qa
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.