4paradigm / 4paradigm/OpenMLDB
ddl parser should return null if sql compile failed
- 主要言語
- C++
- スター
- 1.7k
- フォーク
- 331
- 平均マージ
- 12日 12時間
- マージ済み PR(30日)
- 1
説明
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
コントリビューションガイド
調査の方向性
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.
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- cpp
- 領域
- databases
- issue の種類
- バグ
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 45/100