Apply suggestions from code review

Co-authored-by: Lin Zhihao <59785146+LinZhihao-723@users.noreply.github.com>

Author: sitaowang1998

components/spider-storage/src/state/error.rs

@@ -9,10 +9,10 @@ use crate::{
 #[derive(thiserror::Error, Debug)]
 pub enum StorageServerError {
     #[error(transparent)]
-    Internal(#[from] InternalError),
+    CacheInternal(#[from] InternalError),
 
     #[error(transparent)]
-    StaleState(#[from] StaleStateError),
+    CacheStaleState(#[from] StaleStateError),
 
     #[error("stale session")]
     StaleSession,

components/spider-storage/src/state/job_cache.rs

@@ -11,8 +11,8 @@ use crate::{
 
 /// An in-memory cache for job control blocks.
 ///
-/// This type provides concurrent access to job control blocks via a `DashMap`.
-/// It is generic over the same type parameters as [`SharedJobControlBlock`].
+/// This type provides concurrent access to job control blocks via a `DashMap`. It is generic over
+/// the same type parameters as [`SharedJobControlBlock`].
 ///
 /// # Type Parameters
 ///
@@ -170,7 +170,7 @@ mod tests {
     struct MockDbConnector;
 
     #[async_trait::async_trait]
-    impl crate::db::InternalJobOrchestration for MockDbConnector {
+    impl InternalJobOrchestration for MockDbConnector {
         async fn start(&self, _job_id: JobId) -> Result<(), crate::db::DbError> {
             Ok(())
         }