4paradigm / 4paradigm/OpenMLDB

refactor(physcial pass): adopt `Status` as return code instead of bool

Open
#646 1 comment 1 reaction 2 assignees Claimed by @aceforeverd 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.**
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

**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

The issue is a template with no description of the problem, solution, or context. It mentions refactoring a 'physical pass' to use a 'Status' return code instead of bool. To start, one would need to locate the physical pass code in the OpenMLDB codebase, understand the current bool return pattern, and design a Status type. There is no guidance on files, tests, or what 'done' looks like.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
backend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.