Use 'disabled' as the default of backend_batch_strategy

Author: doyoubi

conf/server-proxy.toml

@@ -12,7 +12,10 @@ slowlog_sample_rate = 1000
 thread_number = 2
 
 backend_conn_num = 2
-backend_batch_strategy = "fixed"
+# "disabled" should be good enough for most cases.
+# For larger throughput with pipline enabled,
+# we can try "fixed" or "dynamic" for higher throughput.
+backend_batch_strategy = "disabled"
 # In bytes
 backend_flush_size = 1024
 # In nanoseconds

src/bin/server_proxy.rs

@@ -65,9 +65,9 @@ fn gen_conf() -> Result<ServerProxyConfig, &'static str> {
             "disabled" => BatchStrategy::Disabled,
             "fixed" => BatchStrategy::Fixed,
             "dynamic" => BatchStrategy::Dynamic,
-            _ => BatchStrategy::Fixed,
+            _ => BatchStrategy::Disabled,
         })
-        .unwrap_or_else(|_| BatchStrategy::Fixed);
+        .unwrap_or_else(|_| BatchStrategy::Disabled);
     let backend_flush_size =
         NonZeroUsize::new(s.get::<usize>("backend_flush_size").unwrap_or(1024))
             .ok_or("backend_flush_size")?;