thomasmueller commented on code in PR #826: URL: https://github.com/apache/jackrabbit-oak/pull/826#discussion_r1082553775
########## oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/flatfile/FlatFileNodeStoreBuilder.java: ########## @@ -222,26 +235,53 @@ public List<FlatFileStore> buildList(IndexHelper indexHelper, IndexerSupport ind return storeList; } - private File createdSortedStoreFile() throws IOException, CompositeException { + /** + * Returns the existing list of store files if it can read from system property OAK_INDEXER_SORTED_FILE_PATH which + * defines the existing folder where the flat file store files are present. Will throw an exception if it cannot + * read or the path in the system property is not a directory. + * If the system property OAK_INDEXER_SORTED_FILE_PATH in undefined, or it cannot read relevant files it + * initializes the flat file store. + * + * @return list of flat files + * @throws IOException + * @throws CompositeException + */ + private List<File> createdSortedStoreFiles() throws IOException, CompositeException { + // Check system property defined path String sortedFilePath = System.getProperty(OAK_INDEXER_SORTED_FILE_PATH); Review Comment: SystemPropertySupplier is nice because it logs if the value is set (to a non-default value). However in this case I think not using it is fine. -- 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: dev-unsubscr...@jackrabbit.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org