4paradigm / 4paradigm/OpenMLDB

hybridse cmake re-organize

Offen
#857 3 Kommentare 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

As hybridse complication was integrated into openmldb. cmake options prefixed with `HYBRIDSE_` to make it clear that they are specific to hybridse.

```
option(TESTING_ENABLE "Enable Test" ON)
option(HYBRIDSE_TESTING_ENABLE "Enable Hybridse Test" ON)
option(EXAMPLES_TESTING_ENABLE "Enable Examples Test" ON)
option(BENCHMARK_ENABLE "Enable Benchmark" OFF)
option(COVERAGE_ENABLE "Enable Coverage" OFF)
option(PYSDK_ENABLE "Enable pysdk" ON)
option(JAVASDK_ENABLE "Enable javasdk" ON)
option(EXAMPLES_ENABLE "Enable examples" ON)
option(LLVM_EXT_ENABLE "Enable llvm ext sources" OFF)
option(SANITIZER_ENABLE "Enable AddressSanitizer in Debug mode" OFF)
```

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

The issue is about renaming CMake options to have a 'HYBRIDSE_' prefix. Look for CMakeLists.txt files in the hybridse directory. Identify all options that need renaming, update their definitions, and update any references in the code or tests. Run the CMake configuration to ensure the build still works.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cmake, cpp
Bereich
build-system
Issue-Typ
Refactoring
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
40/100

Neue Issues direkt in Ihr Postfach

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