apache / apache/datafusion

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

Aperta
#18,878 4 commenti 0 reazioni 0 assegnatari Vedi su GitHub
enhancement
Lingua principale
Rust
Stelle
9.3k
Fork
2.4k
Merge medio
3g 11h
PR unite (30g)
362

Descrizione

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

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Riproduci la discrepanza da datafusion/sqllogictest/test_files/slt_features.slt alla riga 84, quindi traccia le posizioni separate di validazione e rendering degli errori di sqllogictest menzionate nell’issue. La modifica completata dovrebbe sopprimere il diff falso positivo causato da continuando comunque a segnalare la discrepanza reale, con coverage per questo output.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
rust
Ambito
testing-qa
Tipo di issue
Funzionalità
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.