Skip to content

Commit 64834ed

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 38d2fe7 commit 64834ed

5 files changed

Lines changed: 1311 additions & 1632 deletions

File tree

.github/workflows/sql-benchmarks.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -127,6 +127,13 @@ jobs:
127127
128128
- uses: ./.github/actions/system-info
129129

130+
- name: Install ClickHouse
131+
if: contains(matrix.targets, 'clickhouse:')
132+
run: |
133+
curl https://clickhouse.com/ | sh
134+
sudo ./clickhouse install
135+
echo "CLICKHOUSE_BINARY=$(which clickhouse)" >> $GITHUB_ENV
136+
130137
- name: Build binaries
131138
shell: bash
132139
env:

0 commit comments

Comments
 (0)