4paradigm / 4paradigm/OpenMLDB
feat: improve error message system.
- Lenguaje dominante
- C++
- Estrellas
- 1.7k
- Forks
- 331
- Merge medio
- 12 d 12 h
- PR fusionados (30 d)
- 1
Descripción
**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.
Guía de contribución
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- cpp
- Área
- backend, developer-experience
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 20/100