uschindler commented on code in PR #15864:
URL: https://github.com/apache/lucene/pull/15864#discussion_r2979564578
##########
lucene/core/src/test/org/apache/lucene/internal/tests/TestTestSecrets.java:
##########
@@ -37,4 +46,70 @@ public void testCannotSet() {
expectThrows(AssertionError.class, () ->
TestSecrets.setIndexPackageAccess(null));
expectThrows(AssertionError.class, () ->
TestSecrets.setSegmentReaderAccess(null));
}
+
+ public void testDeadlock() throws Exception {
Review Comment:
I think we can add this case there in the thread:
- update switch statement and add a new case that loads TestSecrets and
calls one of the getter methods
- raise the threadcount as described at beginning of tests (2 ztimes the
module/number of cases). I'd improve the test to use a common variable for
that, it is a bit tricky
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
