mikemccand commented on a change in pull request #1848: URL: https://github.com/apache/lucene-solr/pull/1848#discussion_r485498630
########## File path: lucene/core/src/java/org/apache/lucene/index/DefaultIndexingChain.java ########## @@ -73,21 +78,38 @@ // Holds fields seen in each document private PerField[] fields = new PerField[1]; private final InfoStream infoStream; - - DefaultIndexingChain(DocumentsWriterPerThread docWriter) { - this.docWriter = docWriter; - this.fieldInfos = docWriter.getFieldInfosBuilder(); - this.infoStream = docWriter.getIndexWriterConfig().getInfoStream(); + private final ByteBlockPool.Allocator byteBlockAllocator; + private final LiveIndexWriterConfig indexWriterConfig; + private final int indexCreatedVersionMajor; + private final Consumer<Throwable> abortingExceptionConsumer; + private boolean hasHitAbortingException = false; Review comment: Don't need the `= false` -- it is java's default already. ---------------------------------------------------------------- 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. 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