4paradigm / 4paradigm/OpenMLDB

refactor `sql_cmd_test` & `mini_cluster`

Open
#1,383 3 comments 0 reactions 1 assignee Claimed by @aceforeverd View on GitHub
enhancement
Dominant language
C++
Stars
1.7k
Forks
331
Avg merge
12d 12h
Merged PRs (30d)
1

Description

the overall structure of `sql_cmd_test` & `mini_cluster` is a mess, mix of global variables and reference over different test case.

It's hard to maintain and add new test case easily

Contributor guide

Open the contributing guide

Research direction

The issue points to sql_cmd_test and mini_cluster files, likely in a test directory. Start by examining the current structure, global variables, and cross-references between test cases. Understand how tests interact with the mini_cluster. Refactoring involves cleaning up dependencies and improving modularity. Verify changes by running existing tests to ensure they still pass.

Written by the indexing model from the issue text.

Assessment

Domain
testing
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.