bugfix: call bootstrap_db before calling connect

`bootstrap_db` calls the `export` endpoint and bootstraps the db
quickly. If we call `connect` before calling `bootstrap_db`, it will
create a local `.db` file and sync falls back to incremental
bootstrap which is super slow

Author: avinassh

libsql/src/database/builder.rs

@@ -2,9 +2,9 @@ cfg_core! {
     use crate::EncryptionConfig;
 }
 
-use crate::{Database, Result};
-
 use super::DbType;
+use crate::sync::bootstrap_db;
+use crate::{Database, Result};
 
 /// A builder for [`Database`]. This struct can be used to build
 /// all variants of [`Database`]. These variants include:
@@ -646,9 +646,16 @@ cfg_sync! {
             let mut bg_abort: Option<std::sync::Arc<crate::sync::DropAbort>> = None;
 
             if let Some(sync_interval) = sync_interval {
-                let conn = db.connect()?;
                 let sync_ctx = db.sync_ctx.as_ref().unwrap().clone();
+                {
+                    let mut ctx = sync_ctx.lock().await;
+                    bootstrap_db(&mut ctx).await?;
+                }
 
+                // db.connect creates a local db file, so it is important that we always call
+                // `bootstrap_db` (for synced dbs) before calling connect. Otherwise, the sync
+                // protocol skips calling `export` endpoint causing slowdown in initial bootstrap.
+                let conn = db.connect()?;
                 let jh = tokio::spawn(
                     async move {
                         loop {