4paradigm / 4paradigm/OpenMLDB

hybridse cmake re-organize

Ouverte
#857 3 commentaires 0 réactions 1 personne assignée Réclamée par @aceforeverd Voir sur GitHub
enhancement
Langage dominant
C++
Étoiles
1.7k
Forks
331
Merge moyen
12 j 12 h
PR mergées (30 j)
1

Description

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)
```

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

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.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cmake, cpp
Domaine
build-system
Type d'issue
Refactorisation
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
40/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.