uschindler commented on code in PR #15002:
URL: https://github.com/apache/lucene/pull/15002#discussion_r2239174129
##########
lucene/core/src/test/org/apache/lucene/index/TestReaderClosed.java:
##########
@@ -105,9 +105,7 @@ public void testReaderChaining() throws Exception {
if (ace == null) {
throw new AssertionError("Query failed, but not due to an
AlreadyClosedException", e);
}
- assertEquals(
- "this IndexReader cannot be used anymore as one of its child readers
was closed",
- ace.getMessage());
+ assertEquals("this IndexReader is closed", ace.getMessage());
Review Comment:
I am not sure if this exact message is thrown by all directory
implementations. Maybe just check for the type of exection and ignore message.
So just remove this line.
See the test name "testReaderChaining", maybe remove it completely?
--
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
