4paradigm / 4paradigm/OpenMLDB
support execute multi sql in ExecuteSQL
- Dominant language
- C++
- Stars
- 1.7k
- Forks
- 331
- Avg merge
- 12d 12h
- Merged PRs (30d)
- 1
Description
https://github.com/4paradigm/OpenMLDB/blob/06255ca9620adf0e0033411984081dab6906a79f/src/sdk/sql_cluster_router.cc#L2308
When we do `execute("sql1;sql2")`, only sql1 will be executed, and no warning about sql2.
We should support execute multi sql in `ExecuteSQL`
Contributor guide
Research direction
The issue points to sql_cluster_router.cc line 2308 where ExecuteSQL handles SQL strings. Start by examining that function to see how it parses and executes SQL. Look for existing parsing logic or splitting on semicolons. Check if there are tests for ExecuteSQL to understand the expected behavior. 'Done' means the function executes all statements in a semicolon-separated string and possibly returns warnings or errors for each.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- backend, databases
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 45/100