4paradigm / 4paradigm/OpenMLDB

feat: improve error message system.

オープン
#406 コメント 2 件 リアクション 1 件 担当者 1 名 @aceforeverd が担当を希望しています GitHub で見る
enhancement execute-engine
主要言語
C++
スター
1.7k
フォーク
331
平均マージ
12日 12時間
マージ済み PR(30日)
1

説明

**Is your feature request related to a problem? Please describe.**

Users can get frustrated when they use `OpenMLDB` in the wrong way. e.g. SQL Syntax errors, unsupported data types, unsupported query on specific running mode. What's worse, the error message can be misleading in some cases which might make users more confused.
So it's very important to improve, _refactor if need,_ the whole error message system in a way to make the error message clear and readable.

**Describe the solution you'd like**

- [ ] support ERROR CODE to identify the error types.
- [ ] error message had better follow some pattern.
- [x] trace msg can be `Exposed` when the DB enable `TRACE`
- [ ] Move hybridse::base::Status to openmldb::base::

**Describe alternatives you've considered**
A clear and concise description of any alternative solutions or features you've considered.

**Additional context**
Add any other context or screenshots about the feature request here.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

The issue mentions moving hybridse::base::Status to openmldb::base:: and adding error codes. Look at the current error handling in the codebase, especially around SQL parsing and execution. Start by examining the status classes and error message generation. Determine what patterns to follow and how to expose trace messages when TRACE is enabled.

索引モデルが issue の本文から書いたものです。

評価

技術スタック
cpp
領域
backend, developer-experience
issue の種類
機能追加
難易度
5/5
見積もり時間
1週間以上
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
20/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。