4paradigm / 4paradigm/OpenMLDB
hybridse cmake re-organize
- 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
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