Skip to content

v0.2.50..v0.2.51 changeset Testing.conf

Garret Voltz edited this page Jan 15, 2020 · 1 revision
diff --git a/conf/core/Testing.conf b/conf/core/Testing.conf
index 0c27e99..82bb4b1 100644
--- a/conf/core/Testing.conf
+++ b/conf/core/Testing.conf
@@ -7,8 +7,12 @@
   "conflate.tag.disable.value.truncation": "true",
   "#" : "Options required for unit test runs",
   "hash.seed.zero": "true",
-  "#" : "Poi/Polygon address matching has a slow init time that we dont want to incur for every test. We only want to incur it for those",
-  "#" : "on which it is required, and it is enabled selectively in child confs.",
+  "#" : "Address matching has a slow init time incurred on a per test process basis. So, we only want to take that hit for those",
+  "#" : "tests where it is absolutely required. For unit tests, pass the option into the class being tested. For case tests, enable it", 
+  "#" : "selectively in child confs. Since regression tests all use this conf, you'll want to explicitly override these settings in all",
+  "#" : "of them so we have an acccurate picture of address matching's impact on conflation scores. Given the longer runtimes of the",
+  "#" : "regression tests, the performance hit isn't that big of a deal.",
+  "building.address.match.enabled": "false",
   "poi.polygon.address.match.enabled": "false",
   "#" : "At user request in the UI we have hardcoded POI/Polygon tag merging to use the PreserveTypesTagMerger, as there is currently",
   "#" : "no way to dynamically change tag merging from within the UI. Since PreserveTypesTagMerger is still experimental, we want to leave",
Clone this wiki locally