pitrou commented on issue #51095:
URL: https://github.com/apache/arrow/issues/51095#issuecomment-5476974965
What we could do perhaps is only print coredumps on error. This would not
eliminate a potential race condition in `find`, but would make it much less
likely:
```diff
diff --git a/cpp/build-support/run-test.sh b/cpp/build-support/run-test.sh
index 2dc18860a1..c618e90c98 100755
--- a/cpp/build-support/run-test.sh
+++ b/cpp/build-support/run-test.sh
@@ -229,18 +229,16 @@ for ATTEMPT_NUMBER in $(seq 1
"$TEST_EXECUTION_ATTEMPTS") ; do
fi
if [ "$STATUS" -eq "0" ]; then
break
- elif [ "$ATTEMPT_NUMBER" -lt "$TEST_EXECUTION_ATTEMPTS" ]; then
- echo Test failed attempt number "$ATTEMPT_NUMBER"
- echo Will retry...
fi
+ print_coredumps
+ echo Test failed attempt number "$ATTEMPT_NUMBER"
+ echo Will retry...
done
if [ "$RUN_TYPE" = "test" ]; then
post_process_tests
fi
-print_coredumps
-
popd
rm -Rf "$TEST_WORKDIR"
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
