NVIDIA / NVIDIA/cudf

[BUG] [JNI] Switch from `assert` to checked enabled at runtime

Open
#22,227 0 comments 0 reactions 0 assignees View on GitHub
bug Java
Dominant language
C++
Stars
9.8k
Forks
1.1k
Avg merge
3d 6m
Merged PRs (30d)
278

Description

The invariant checks and parameter checks in `ColumnVector.java` are currently checked using `assert`. The problem here is that these asserts only fire when enabled (`-ea`).

It would be good to switch these to checks that are always enabled at runtime.
Something like [`Preconditions`](https://github.com/NVIDIA/spark-rapids-jni/commits/main/src/main/java/com/nvidia/spark/rapids/jni/Preconditions.java) come to mind. But that is specific to `spark-rapids-jni`.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.