4paradigm / 4paradigm/OpenMLDB

Add more assertions for cluster(distribution) mode

Open
#632 0 comments 0 reactions 1 assignee 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 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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.