Skip to content

Commit 01f3ab7

Browse files
committed
bench(clickhouse): remove build-time binary download, resolve clickhouse from PATH
- Remove reqwest-based binary download from build.rs - Resolve clickhouse binary via CLICKHOUSE_BINARY env var or $PATH at runtime - Add CI step to install clickhouse before building when needed - Fail with clear error message if binary is not found locally Signed-off-by: fastio <pengjian.uestc@gmail.com>
1 parent 57ba14a commit 01f3ab7

5 files changed

Lines changed: 1286 additions & 1097 deletions

File tree

.github/workflows/sql-benchmarks.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -135,6 +135,13 @@ jobs:
135135
136136
- uses: ./.github/actions/system-info
137137

138+
- name: Install ClickHouse
139+
if: contains(matrix.targets, 'clickhouse:')
140+
run: |
141+
curl https://clickhouse.com/ | sh
142+
sudo ./clickhouse install
143+
echo "CLICKHOUSE_BINARY=$(which clickhouse)" >> $GITHUB_ENV
144+
138145
- name: Build binaries
139146
shell: bash
140147
env:

0 commit comments

Comments
 (0)