4paradigm / 4paradigm/OpenMLDB

ddl parser should return null if sql compile failed

Abierto
#3,890 0 comentarios 0 reacciones 1 asignado Reclamado por @vagetablechicken Ver en GitHub
bug
Lenguaje dominante
C++
Estrellas
1.7k
Forks
331
Merge medio
12 d 12 h
PR fusionados (30 d)
1

Descripción

https://github.com/4paradigm/OpenMLDB/blob/e1369fb137602621bd41674077c5ab1c46b2d505/src/base/ddl_parser.cc#L122

We should check return value and status. Even the sql compile failed, the map `index_hints` may still have some index hints.

And it effect OpenMLDBSQLEmulator, release a new version, v1.1-beta

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

The issue points to src/base/ddl_parser.cc line 122. Check the return value and status handling in the DDL parser to ensure that if SQL compilation fails, the function returns null and does not leave partial data in the index_hints map. Review the surrounding code to understand the error flow, then verify the fix by testing with OpenMLDBSQLEmulator.

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

Evaluación

Stack tecnológico
cpp
Área
databases
Tipo de issue
Error
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.