4paradigm / 4paradigm/OpenMLDB
Add more assertions for cluster(distribution) mode
- 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 cluster mode, performing the Last join query without index optimization (enable_distsql=true) is not allowable. So, the engine should throw a compile error when we turn on enable_distsql instead of running with wrong results.
This issue is going to add more assertions in cluster mode to avoid bugs like #612
**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.
Contributor guide
Research direction
Look at issue #612 for context on the bug. The work involves adding assertions in cluster mode, likely in the query compiler or planner when enable_distsql is true. Search for 'enable_distsql' and 'Last join' in the codebase to find relevant files. The goal is to throw a compile error instead of producing wrong results. Run existing tests related to cluster mode and distsql to ensure the new assertions work.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- sql
- Domain
- backend, databases, machine-learning
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100