4paradigm / 4paradigm/OpenMLDB

ddl parser should return null if sql compile failed

Ouverte
#3,890 0 commentaires 0 réactions 1 personne assignée Réclamée par @vagetablechicken Voir sur GitHub
bug
Langage dominant
C++
Étoiles
1.7k
Forks
331
Merge moyen
12 j 12 h
PR mergées (30 j)
1

Description

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

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

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.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cpp
Domaine
databases
Type d'issue
Bug
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.