rmuir commented on issue #11755:
URL: https://github.com/apache/lucene/issues/11755#issuecomment-1240123892
Maybe i'm being overly picky, if you look at the rest of the tests.
This test is named `testAddDocumentOnDiskFull` and looks for an ACE
(IOException) on addDocument(). But rarely here, you can get
IllegalStateException when it calls commit().
testAddIndexOnDiskFull() has `catch (IllegalStateException | IOException |
MergePolicy.MergeException e) {`, so its ok with any of these varieties.
We can fix the test to stop failing to reflect reality at least as a small
step.
Happy to open a followup-issue to see if we can improve the
exception-handling to be more consistent.
--
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]
