4paradigm / 4paradigm/OpenMLDB

feat: refactor index option and move index option validation code to `SqlClusterRouter`

Offen
#220 0 Kommentare 0 Reaktionen 1 zugewiesene Person Beansprucht von @jingchen2222 Auf GitHub ansehen
call-for-contributions enhancement
Vorherrschende Sprache
C++
Sterne
1.7k
Forks
331
Ø Merge
12 T. 12 Std.
Gemergte PRs (30 T.)
1

Beschreibung

**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 [...]

Index key, ttl, ttl_type verify logic should be placed in `DB` or in `CORE`

**Describe the solution you'd like**
A clear and concise description of what you want to happen.

1. Refeactor ColumnIndexNode. We only keep general structure.
2. Move `ttl`, `ttl_type`, `verson` logic to `OPENMLDB`
**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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Look at the ColumnIndexNode structure and the SqlClusterRouter class to understand the current index option validation logic. Determine which parts are general and which are specific to OPENMLDB. Move ttl, ttl_type, and version validation to the appropriate location, ensuring the refactored code maintains functionality.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Bereich
database, machine-learning
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.