apache / apache/datafusion

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

Abierto
#18,878 4 comentarios 0 reacciones 0 asignados Ver en GitHub
enhancement
Lenguaje dominante
Rust
Estrellas
9.3k
Forks
2.4k
Merge medio
3 d 11 h
PR fusionados (30 d)
360

Descripción

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

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Reproduce la discrepancia de datafusion/sqllogictest/test_files/slt_features.slt en la línea 84 y, después, sigue las ubicaciones independientes de validación y representación de errores de sqllogictest mencionadas en el issue. El cambio completado debería suprimir el diff falso positivo causado por y seguir informando de la discrepancia real, con cobertura para esta salida.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
rust
Área
testing-qa
Tipo de issue
Nueva funcionalidad
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.