tests: cache symbols ahead of time (tikv#10994)

* tests: cache symbols ahead of time TiKV uses backtrace to record suspicious problems in tests. But creating a backtrace can take several seconds and lead to unexpected timeout for cases. This PR loads all symbols ahead of time to reduce the probability of timeout. tikv-server binary also uses similar way to get around the issue. Signed-off-by: Jay Lee <[email protected]> * fix format Signed-off-by: Jay Lee <[email protected]> Co-authored-by: qupeng <[email protected]> Co-authored-by: Ti Chi Robot <[email protected]>
sunshinejiali · Sep 27, 2021 · f9bbdbc · f9bbdbc
1 parent 00f4586
commit f9bbdbc
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 3 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/components/test_raftstore/src/util.rs b/components/test_raftstore/src/util.rs
@@ -373,6 +373,8 @@ impl Drop for CallbackLeakDetector {
         if self.called {
             return;
         }
+
+        debug!("before capture");
         let bt = backtrace::Backtrace::new();
         warn!("callback is dropped"; "backtrace" => ?bt);
     }

diff --git a/components/test_util/Cargo.toml b/components/test_util/Cargo.toml
@@ -24,6 +24,7 @@ cloud-aws = [ "encryption_export/cloud-aws" ]
 cloud-gcp = [ "encryption_export/cloud-gcp" ]
 
 [dependencies]
+backtrace = "0.3"
 encryption_export = { path = "../encryption/export", default-features = false }
 fail = "0.4"
 grpcio = { version = "0.9",  default-features = false, features = ["openssl-vendored"] }

diff --git a/components/test_util/src/lib.rs b/components/test_util/src/lib.rs
@@ -14,8 +14,8 @@ mod runner;
 mod security;
 
 use rand::Rng;
-use std::env;
 use std::sync::atomic::{AtomicU16, Ordering};
+use std::{env, thread};
 
 pub use crate::encryption::*;
 pub use crate::kv_generator::*;
@@ -27,6 +27,14 @@ pub use crate::runner::{
 pub use crate::security::*;
 
 pub fn setup_for_ci() {
+    // We use backtrace in tests to record suspicious problems. And loading backtrace
+    // the first time can take several seconds. Spawning a thread and load it ahead
+    // of time to avoid causing timeout.
+    thread::Builder::new()
+        .name(tikv_util::thd_name!("backtrace-loader"))
+        .spawn(::backtrace::Backtrace::new)
+        .unwrap();
+
     if env::var("CI").is_ok() {
         // HACK! Use `epollex` as the polling engine for gRPC when running CI tests on
         // Linux and it hasn't been set before.