mikemccand commented on code in PR #15885:
URL: https://github.com/apache/lucene/pull/15885#discussion_r3015728261


##########
lucene/core/src/java/org/apache/lucene/index/IndexWriter.java:
##########
@@ -2844,7 +2844,7 @@ private synchronized void publishFlushedSegment(
       ensureOpen(false);
 
       if (infoStream.isEnabled("IW")) {
-        infoStream.message("IW", "publishFlushedSegment " + newSegment);

Review Comment:
   Oh, can you also fix `IndexWriter` init, if it sees an existing index when 
it starts up, to log all segments with `toStringVerbose()`?  This is important 
for segment tracing, so if it's given such an `InfoStream` (that did not 
witness the birth of the index) it can know specifics about the pre-existing 
segments.
   
   I will tests the next iteration of this PR by generating segment trace for 
luceneutil indexing!



-- 
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]

Reply via email to