feat: use RegionNotReady instead of RegionNotFound for a registering …

…region
GreptimeTeam · Dec 20, 2023 · c3a9c3d · c3a9c3d
1 parent e6dac4a
commit c3a9c3d
Show file tree

Hide file tree

Showing 5 changed files with 366 additions and 36 deletions.
diff --git a/src/datanode/src/error.rs b/src/datanode/src/error.rs
@@ -210,6 +210,12 @@ pub enum Error {
         location: Location,
     },
 
+    #[snafu(display("RegionId {} not ready", region_id))]
+    RegionNotReady {
+        region_id: RegionId,
+        location: Location,
+    },
+
     #[snafu(display("Region engine {} is not registered", name))]
     RegionEngineNotFound { name: String, location: Location },
 
@@ -295,6 +301,7 @@ impl ErrorExt for Error {
             | GetRegionMetadata { .. } => StatusCode::Internal,
 
             RegionNotFound { .. } => StatusCode::RegionNotFound,
+            RegionNotReady { .. } => StatusCode::RegionNotReady,
 
             StartServer { source, .. } | ShutdownServer { source, .. } => source.status_code(),