mikemccand opened a new pull request, #12535: URL: https://github.com/apache/lucene/pull/12535
Spinoff from [this shocking build failure](https://lists.apache.org/thread/t79ctdmql0j7v4s8gd9ovk6z7lzrl0w9). The OS will pick a free socket address to bind to since we pass `0`, and we tell the clients to use the assigned port already, so we don't need to ask the OS to reuse the local address. Also remove the 30 second timeout: the test can wait for crazy slow clients to wake up and connect. I confirmed I could still run the `LockVerifyServer` and two `LockStressTest` clients from the command-line, testing `NIOFSLockFactory`. Fortunately, it passed. Maybe we should move all this lock verifying stuff out of `core`? I don't think this is a tool users often use? -- 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: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
