4paradigm / 4paradigm/OpenMLDB

Add dependent table to `hybridse::vm::ExplainOutput`

Open
#607 1 comment 0 reactions 2 assignees Claimed by @jingchen2222 View on GitHub
enhancement
Dominant language
C++
Stars
1.7k
Forks
331
Avg merge
12d 12h
Merged PRs (30d)
1

Description

**Is your feature request related to a problem? Please describe.**

In order to query SQL, we must call `Explain` to obtain the `ExlainInfo` and call `GetDependentTables` to get dependent tables.
In this issue, we are going to add dependent tables info into `ExlainInfo`.

**Describe the solution you'd like**

**Describe alternatives you've considered**

**Additional context**

Contributor guide

Open the contributing guide

Research direction

Look at the hybridse::vm::ExplainOutput structure and the GetDependentTables function to understand how dependent tables are currently obtained. Modify ExplainOutput to include dependent tables information, ensuring the change is consistent across the codebase. Test by running existing SQL query tests to verify the new info is correctly populated.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
databases, machine-learning
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.