More PR comments addressed

downstairs/src/main.rs

@@ -260,16 +260,6 @@ async fn main() -> Result<()> {
             source,
             trace_endpoint,
         } => {
-            /*
-             * If any of our async tasks in our runtime panic, then we should
-             * exit the program right away.
-             */
-            let default_panic = std::panic::take_hook();
-            std::panic::set_hook(Box::new(move |info| {
-                default_panic(info);
-                std::process::exit(1);
-            }));
-
             // Instrumentation is shared.
             if let Some(endpoint) = trace_endpoint {
                 let tracer = opentelemetry_jaeger::new_agent_pipeline()

downstairs/src/repair.rs

@@ -70,16 +70,7 @@ pub async fn repair_main(
     let ds = ds.lock().await;
     let region_dir = ds.region.dir.clone();
     let read_only = ds.read_only;
-    let rd = region::config_path(region_dir.clone());
-    let region_definition = match read_json(&rd) {
-        Ok(def) => def,
-        Err(e) => {
-            return Err(format!(
-                "Can't get region definition from {:?}: {:?}",
-                rd, e
-            ));
-        }
-    };
+    let region_definition = ds.region.def();
     drop(ds);
 
     info!(log, "Repair listens on {} for path:{:?}", addr, region_dir);