4paradigm / 4paradigm/OpenMLDB

hybridse cmake re-organize

オープン
#857 コメント 3 件 リアクション 0 件 担当者 1 名 @aceforeverd が担当を希望しています GitHub で見る
enhancement
主要言語
C++
スター
1.7k
フォーク
331
平均マージ
12日 12時間
マージ済み PR(30日)
1

説明

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

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

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.

索引モデルが issue の本文から書いたものです。

評価

技術スタック
cmake, cpp
領域
build-system
issue の種類
リファクタリング
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
40/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。