Improve error message in `sqllogictest` with `<slt:ignore>` marker
- 主要言語
- Rust
- スター
- 9.3k
- フォーク
- 2.4k
- 平均マージ
- 3日 11時間
- マージ済み PR(30日)
- 360
説明
### 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_
コントリビューションガイド
調査の方向性
datafusion/sqllogictest/test_files/slt_features.slt の84行目にある不一致を再現し、次に issue で言及されている sqllogictest の検証箇所とエラーレンダリング箇所をそれぞれ追跡します。完成した変更では、 によって生じる誤検知の diff を抑制しつつ、実際の不一致は引き続き報告する必要があります。この出力に対する coverage も追加します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- rust
- 領域
- testing-qa
- issue の種類
- 機能追加
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 45/100