Skip to content

Commit

Permalink
Clarify source of error message in stream testing. (rapidsai#13541)
Browse files Browse the repository at this point in the history
This PR changes the error message from stream testing to identify that it's being raised by a cudf test utility. I didn't recognize the error text at first, and assumed it was an error from CUDA or nvbench.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - David Wendt (https://github.com/davidwendt)
  - Vyas Ramasubramani (https://github.com/vyasr)

URL: rapidsai#13541
  • Loading branch information
bdice authored Jun 9, 2023
1 parent 021854b commit be501f5
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions cpp/tests/utilities/identify_stream_usage.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -161,9 +161,9 @@ void check_stream_and_error(cudaStream_t stream)
#endif // __GNUC__
char const* env_stream_error_mode{std::getenv("GTEST_CUDF_STREAM_ERROR_MODE")};
if (env_stream_error_mode && !strcmp(env_stream_error_mode, "print")) {
std::cout << "Found unexpected stream!" << std::endl;
std::cout << "cudf_identify_stream_usage found unexpected stream!" << std::endl;
} else {
throw std::runtime_error("Found unexpected stream!");
throw std::runtime_error("cudf_identify_stream_usage found unexpected stream!");
}
}
}
Expand Down

0 comments on commit be501f5

Please sign in to comment.