-
Notifications
You must be signed in to change notification settings - Fork 74
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",