add checks for history related tunables

vladak · vladak · commit 3372675c9ce8 · 2022-05-24T18:20:18.000+02:00
diff --git a/opengrok-indexer/src/main/java/org/opengrok/indexer/index/Indexer.java b/opengrok-indexer/src/main/java/org/opengrok/indexer/index/Indexer.java
@@ -908,6 +908,14 @@ private static void checkConfiguration() {
         if (!new File(cfg.getDataRoot()).canWrite()) {
             die("Data root '" + cfg.getDataRoot() + "' must be writable");
         }
+
+        if (!cfg.isHistoryEnabled() && cfg.isHistoryBasedReindex()) {
+            die("History has to be enabled for history based reindex");
+        }
+
+        if (!cfg.isHistoryCache() && cfg.isHistoryBasedReindex()) {
+            die("History cache has to be enabled for history based reindex");
+        }
     }
 
     private static void die(String message) {

Original file line number	Diff line number	Diff line change
`@@ -908,6 +908,14 @@ private static void checkConfiguration() {`
`908`	`908`	`if (!new File(cfg.getDataRoot()).canWrite()) {`
`909`	`909`	`die("Data root '" + cfg.getDataRoot() + "' must be writable");`
`910`	`910`	`}`
	`911`	`+`
	`912`	`+ if (!cfg.isHistoryEnabled() && cfg.isHistoryBasedReindex()) {`
	`913`	`+ die("History has to be enabled for history based reindex");`
	`914`	`+ }`
	`915`	`+`
	`916`	`+ if (!cfg.isHistoryCache() && cfg.isHistoryBasedReindex()) {`
	`917`	`+ die("History cache has to be enabled for history based reindex");`
	`918`	`+ }`
`911`	`919`	`}`
`912`	`920`
`913`	`921`	`private static void die(String message) {`