Skip to content

Commit 1b9a3ad

Browse files
committed
ci: merge sanitizer jobs into build matrix
1 parent 526e44f commit 1b9a3ad

2 files changed

Lines changed: 10 additions & 83 deletions

File tree

.github/workflows/build_and_test.yaml

Lines changed: 10 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -41,43 +41,41 @@ jobs:
4141
- name: gcc-release
4242
cc: gcc-14
4343
cxx: g++-14
44-
fetch_depth: 1
4544
build_args: --build_type Release
4645
- name: clang-release
47-
cc: clang
48-
cxx: clang++
49-
fetch_depth: 1
5046
build_args: --build_type Release
5147
- name: gcc-debug
5248
cc: gcc-14
5349
cxx: g++-14
54-
fetch_depth: 1
55-
build_args: ''
5650
- name: clang-debug
57-
cc: clang
58-
cxx: clang++
59-
fetch_depth: 0 # fetch the PR target branch history for clang-tidy
51+
fetch_depth: '0' # fetch the PR target branch history for clang-tidy
6052
build_args: >-
6153
--check_clang_tidy
6254
--lint_git_target_commit "origin/${{ github.base_ref || github.event.repository.default_branch }}"
55+
- name: asan-ubsan
56+
build_args: --enable_asan --enable_ubsan
57+
- name: tsan
58+
skip_rust: true
59+
build_args: --enable_tsan
6360
steps:
6461
- name: Checkout paimon-cpp
6562
uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
6663
with:
6764
lfs: true
68-
fetch-depth: ${{ matrix.fetch_depth }}
65+
fetch-depth: ${{ matrix.fetch_depth || '1' }}
6966
- name: Setup ccache
7067
uses: ./.github/actions/setup-ccache
7168
with:
7269
cache-key-prefix: ccache-${{ matrix.name }}
7370
- name: Install Rust toolchain (tantivy-fts)
71+
if: ${{ !matrix.skip_rust }}
7472
shell: bash
7573
run: ci/scripts/setup_rust.sh
7674
- name: Build Paimon
7775
shell: bash
7876
env:
79-
CC: ${{ matrix.cc }}
80-
CXX: ${{ matrix.cxx }}
77+
CC: ${{ matrix.cc || 'clang' }}
78+
CXX: ${{ matrix.cxx || 'clang++' }}
8179
run: >-
8280
ci/scripts/build_paimon.sh
8381
--source_dir "$(pwd)"

.github/workflows/test_with_sanitizer.yaml

Lines changed: 0 additions & 71 deletions
This file was deleted.

0 commit comments

Comments
 (0)