From 384f84f30cc7116197a2c03fee7a4ee6f783ccc6 Mon Sep 17 00:00:00 2001
From: Devdutt Shenoi <devdutt@parseable.com>
Date: Thu, 5 Dec 2024 19:10:46 +0530
Subject: [PATCH] `ObjectStorage: Debug`

---
 src/storage/azure_blob.rs     | 1 +
 src/storage/localfs.rs        | 1 +
 src/storage/object_storage.rs | 2 +-
 src/storage/s3.rs             | 1 +
 4 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/storage/azure_blob.rs b/src/storage/azure_blob.rs
index 0f3099585..2493d3d6f 100644
--- a/src/storage/azure_blob.rs
+++ b/src/storage/azure_blob.rs
@@ -191,6 +191,7 @@ pub fn to_object_store_path(path: &RelativePath) -> StorePath {
 
 // ObjStoreClient is generic client to enable interactions with different cloudprovider's
 // object store such as S3 and Azure Blob
+#[derive(Debug)]
 pub struct BlobStore {
     client: LimitStore<MicrosoftAzure>,
     account: String,
diff --git a/src/storage/localfs.rs b/src/storage/localfs.rs
index b3d3e09cd..31f1a62f2 100644
--- a/src/storage/localfs.rs
+++ b/src/storage/localfs.rs
@@ -80,6 +80,7 @@ impl ObjectStorageProvider for FSConfig {
     }
 }
 
+#[derive(Debug)]
 pub struct LocalFS {
     // absolute path of the data directory
     root: PathBuf,
diff --git a/src/storage/object_storage.rs b/src/storage/object_storage.rs
index e70c326bd..443dd930a 100644
--- a/src/storage/object_storage.rs
+++ b/src/storage/object_storage.rs
@@ -67,7 +67,7 @@ pub trait ObjectStorageProvider: StorageMetrics + std::fmt::Debug + Send + Sync
 }
 
 #[async_trait]
-pub trait ObjectStorage: Send + Sync + 'static {
+pub trait ObjectStorage: std::fmt::Debug + Send + Sync + 'static {
     async fn get_object(&self, path: &RelativePath) -> Result<Bytes, ObjectStorageError>;
     // TODO: make the filter function optional as we may want to get all objects
     async fn get_objects(
diff --git a/src/storage/s3.rs b/src/storage/s3.rs
index a501c6250..cc2ccfdca 100644
--- a/src/storage/s3.rs
+++ b/src/storage/s3.rs
@@ -316,6 +316,7 @@ fn to_object_store_path(path: &RelativePath) -> StorePath {
     StorePath::from(path.as_str())
 }
 
+#[derive(Debug)]
 pub struct S3 {
     client: LimitStore<AmazonS3>,
     bucket: String,