refactor(storage): Simplify debug output of some structs

src/storage/src/region.rs

@@ -15,7 +15,9 @@
 #[cfg(test)]
 mod tests;
 mod writer;
+
 use std::collections::BTreeMap;
+use std::fmt;
 use std::sync::Arc;
 
 use async_trait::async_trait;
@@ -49,7 +51,6 @@ use crate::wal::Wal;
 use crate::write_batch::WriteBatch;
 
 /// [Region] implementation.
-#[derive(Debug)]
 pub struct RegionImpl<S: LogStore> {
     inner: Arc<RegionInner<S>>,
 }
@@ -62,6 +63,15 @@ impl<S: LogStore> Clone for RegionImpl<S> {
     }
 }
 
+impl<S: LogStore> fmt::Debug for RegionImpl<S> {
+    fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
+        f.debug_struct("RegionImpl")
+            .field("id", &self.inner.shared.id)
+            .field("name", &self.inner.shared.name)
+            .finish()
+    }
+}
+
 #[async_trait]
 impl<S: LogStore> Region for RegionImpl<S> {
     type Error = Error;
@@ -440,7 +450,6 @@ impl SharedData {
 
 pub type SharedDataRef = Arc<SharedData>;
 
-#[derive(Debug)]
 struct RegionInner<S: LogStore> {
     shared: SharedDataRef,
     writer: RegionWriterRef,

src/storage/src/region/tests.rs

@@ -271,6 +271,10 @@ async fn test_new_region() {
 
     assert_eq!(region_name, region.name());
     assert_eq!(expect_schema, *region.in_memory_metadata().schema());
+    assert_eq!(
+        r#"RegionImpl { id: 0, name: "region-0" }"#,
+        format!("{:?}", region)
+    );
 }
 
 #[tokio::test]