4paradigm / 4paradigm/OpenMLDB

feat: support and validate query table with multiple-levels path

オープン
#365 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
enhancement
主要言語
C++
スター
1.7k
フォーク
331
平均マージ
12日 12時間
マージ済み PR(30日)
1

説明

**Is your feature request related to a problem? Please describe.**
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

There are some problems when users try to query a table named by multiple levels.
For example, given a statement like: `SELECT col1, col2 from db1.t1;`. We are expecting:
- there is a table `t1` under database `db1`
- the current execute-engine is running on database `db1`

Similar rules should be applied on `CREATE INDEX ... `, `CREATE PROCEDURE ... ` and other statements as well

**Describe the solution you'd like**
A clear and concise description of what you want to happen.

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

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

この issue は、SELECT、CREATE INDEX、CREATE PROCEDURE ステートメントで `db1.t1` のような複数レベルのパスを持つテーブルをクエリすることに言及しています。codebase 内の SQL parser と validation ロジックを探してください。おそらく src/parser や src/sql のようなディレクトリにあります。execution engine が現在 database context と table resolution をどのように扱っているかを理解する必要があります。解決策は、statement types 全体で複数レベルのパス構文をサポートし強制するように validation を拡張することです。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
sql
領域
backend, databases
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
25/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。