4paradigm / 4paradigm/OpenMLDB

feat: improve error message system.

Offen
#406 2 Kommentare 1 Reaktion 1 zugewiesene Person Beansprucht von @aceforeverd Auf GitHub ansehen
enhancement execute-engine
Vorherrschende Sprache
C++
Sterne
1.7k
Forks
331
Ø Merge
12 T. 12 Std.
Gemergte PRs (30 T.)
1

Beschreibung

**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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

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.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
backend, developer-experience
Issue-Typ
Feature
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
20/100

Neue Issues direkt in Ihr Postfach

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