-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
use parallel for ci_test #94
Conversation
LHT129
commented
Oct 24, 2024
•
edited
Loading
edited
- introduce Makefile new tag: test_asan_parallel
- mark [diskann] & [hnsw] tags run parallel
1ce403b
to
08696f1
Compare
src/bitset_impl_test.cpp
Outdated
@@ -22,6 +22,10 @@ | |||
|
|||
using namespace roaring; | |||
|
|||
TEST_CASE("false") { | |||
REQUIRE(1 == 0); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
force failure this task ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just for test. remove now
0160a2e
to
51172e9
Compare
test_asan_bg.sh
Outdated
exit_codes+=($?) | ||
done | ||
|
||
all_successful=true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
here can direct return exit_codes
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exit codes is an array. we may add some logger to detail which case fail in future
- introduce Makefile new tag: test_asan_parallel - mark [diskann] & [hnsw] tags run parallel Signed-off-by: LHT129 <tianlan.lht@antgroup.com>
lgtm |