4paradigm / 4paradigm/OpenMLDB

deprecating legacy schema codec

Offen
#3,722 0 Kommentare 0 Reaktionen 1 zugewiesene Person Beansprucht von @aceforeverd Auf GitHub ansehen
enhancement sql
Vorherrschende Sprache
C++
Sterne
1.7k
Forks
331
Ø Merge
12 T. 12 Std.
Gemergte PRs (30 T.)
1

Beschreibung

The schema codec in https://github.com/4paradigm/OpenMLDB/blob/ff7e8acf21ead1f9734eef59ac3521adb05dff2e/hybridse/include/codec/fe_schema_codec.h#L34-L110

is not real serialization solution for schemas. It

1. Not extendable. complex data type like #3680 is not possible without breaking change, it can not represent map type in single byte.
2. Hardware dependent. Endianness and primitive type size vary on different platform
3. Lack of type safety check, and is error prone.

## Solution ?

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

The issue points to hybridse/include/codec/fe_schema_codec.h lines 34-110, which contains the legacy schema codec. First, examine the current codec implementation to understand its limitations. Then, look for any existing serialization solutions in the codebase that could replace it. The goal is to design a new, extensible, platform-independent, and type-safe schema serialization method, but the issue does not specify a concrete solution or next steps.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
backend, databases
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
30/100

Neue Issues direkt in Ihr Postfach

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