Skip to content

Commit 79b7afc

Browse files
committed
add CI step to validate cudf compatibility
Signed-off-by: Andrew Duffy <andrew@a10y.dev>
1 parent d376502 commit 79b7afc

File tree

2 files changed

+8
-1
lines changed

2 files changed

+8
-1
lines changed

.github/workflows/ci.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -537,6 +537,13 @@ jobs:
537537
--no-fail-fast \
538538
--target x86_64-unknown-linux-gnu \
539539
--verbose
540+
- name: Build cudf-test library
541+
run: cargo +nightly build --locked -p vortex-cudf-test --target x86_64-unknown-linux-gnu
542+
- name: Download and run cudf-test-harness
543+
run: |
544+
curl -fsSL https://github.com/vortex-data/cudf-test-harness/releases/latest/download/cudf-test-harness-x86_64.tar.gz | tar -xz
545+
cd cudf-test-harness-x86_64
546+
./cudf-test-harness check $GITHUB_WORKSPACE/target/x86_64-unknown-linux-gnu/debug/libvortex_cudf_test.so
540547
541548
rust-test-other:
542549
name: "Rust tests (${{ matrix.os }})"

vortex-cuda/src/session.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ impl CudaSession {
121121
.load_function(module_name, type_suffixes, &self.context)
122122
}
123123

124-
/// Get a handle to the exporter that converts Vortex arrays to [`A`rrowDeviceArray`].
124+
/// Get a handle to the exporter that converts Vortex arrays to [`ArrowDeviceArray`].
125125
pub fn export_device_array(&self) -> &Arc<dyn ExportDeviceArray> {
126126
&self.export_device_array
127127
}

0 commit comments

Comments
 (0)