4paradigm / 4paradigm/OpenMLDB

hybridse cmake re-organize

Open
#857 3 comments 0 reactions 1 assignee Claimed by @aceforeverd View on GitHub
enhancement
Dominant language
C++
Stars
1.7k
Forks
331
Avg merge
12d 12h
Merged PRs (30d)
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)
```

Contributor guide

Open the contributing guide

Research direction

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.

Written by the indexing model from the issue text.

Assessment

Tech stack
cmake, cpp
Domain
build-system
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
40/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.