4paradigm / 4paradigm/OpenMLDB

too many type systems in source code but serves same purpose

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

Beschreibung

- https://github.com/4paradigm/OpenMLDB/blob/ff7e8acf21ead1f9734eef59ac3521adb05dff2e/hybridse/include/node/node_enum.h#L154-L165
- https://github.com/4paradigm/OpenMLDB/blob/ff7e8acf21ead1f9734eef59ac3521adb05dff2e/hybridse/src/proto/fe_type.proto#L21-L27
- https://github.com/4paradigm/OpenMLDB/blob/ff7e8acf21ead1f9734eef59ac3521adb05dff2e/src/proto/type.proto#L25-L32
- `base::StringRef, base::Timestamp` .. etc in udf system
- https://github.com/4paradigm/OpenMLDB/blob/ff7e8acf21ead1f9734eef59ac3521adb05dff2e/hybridse/include/sdk/base_schema.h#L27-L32

And also different implementations build upon those type systems,

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

The issue points to multiple type definitions across hybridse/include/node/node_enum.h, hybridse/src/proto/fe_type.proto, src/proto/type.proto, and base_schema.h. Start by examining each linked file to understand the overlapping type systems. Look for how these types are used in the UDF system and other implementations. Determine a strategy to unify or reduce duplication, which requires deep knowledge of the codebase's architecture.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
backend, databases, machine-learning
Issue-Typ
Refactoring
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
20/100

Neue Issues direkt in Ihr Postfach

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