Skip to content

Commit 2f2dfce

Browse files
authored
ci: use the SQL fixture profile for Vortex conversion (#10728)
Signed-off-by: Robert Kruszewski <github@robertk.io>
1 parent 756f1eb commit 2f2dfce

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

‎vortex-sqllogictest/slt/clickbench/generate_data.sh‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
# `hits` table) to slt/clickbench/data/, converts it to Vortex, and checks that
88
# the Parquet and Vortex files hold identical data before the Vortex file is
99
# used by the SLT suite. Set VORTEX_SLT_PROFILE to reuse an existing cargo
10-
# profile for the parity test (CI uses `ci`).
10+
# profile for conversion and the parity test (CI uses `ci`).
1111

1212
set -e -o pipefail
1313

@@ -28,7 +28,7 @@ fi
2828

2929
echo "Converting hits.parquet to Vortex..."
3030
rm -f "${VORTEX_FILE}"
31-
(cd "${CRATE_DIR}" && cargo run --release --package vortex-tui --bin vx -- convert "${PARQUET_FILE}")
31+
(cd "${CRATE_DIR}" && cargo run --profile "${VORTEX_SLT_PROFILE:-release}" --package vortex-tui --bin vx -- convert "${PARQUET_FILE}")
3232

3333
# The parity test reads both files through DuckDB and fails if any row differs,
3434
# so a bad conversion is caught before the query result files are trusted. The

‎vortex-sqllogictest/slt/tpcds/generate_data.sh‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
# one Parquet file per table, converts every table to Vortex,
88
# and checks that the Parquet and Vortex tables hold identical data before the
99
# Vortex files are used by the SLT suite. Set VORTEX_SLT_PROFILE to reuse an
10-
# existing cargo profile for the parity test (CI uses `ci`).
10+
# existing cargo profile for conversion and the parity test (CI uses `ci`).
1111

1212
set -e -o pipefail
1313

@@ -34,7 +34,7 @@ rm -f "${DATA_DIR}"/dbgen_version.parquet
3434
# can read them.
3535
for f in "${DATA_DIR}"/*.parquet; do
3636
echo "Converting $(basename "$f") to Vortex..."
37-
(cd "${CRATE_DIR}" && cargo run --release --package vortex-tui --bin vx -- convert "$f")
37+
(cd "${CRATE_DIR}" && cargo run --profile "${VORTEX_SLT_PROFILE:-release}" --package vortex-tui --bin vx -- convert "$f")
3838
done
3939

4040
# The parity test reads every table in both formats through DuckDB and fails if

0 commit comments

Comments
 (0)