remove unused error variants

Author: YaroslavLitvinov

crates/api-snowflake-rest/src/server/error.rs

@@ -20,30 +20,6 @@ pub type Result<T> = std::result::Result<T, Error>;
 #[snafu(visibility(pub(crate)))]
 #[error_stack_trace::debug]
 pub enum Error {
-    #[snafu(display("Failed to decompress GZip body"))]
-    GZipDecompress {
-        #[snafu(source)]
-        error: std::io::Error,
-        #[snafu(implicit)]
-        location: Location,
-    },
-
-    #[snafu(display("Failed to parse login request"))]
-    LoginRequestParse {
-        #[snafu(source)]
-        error: serde_json::Error,
-        #[snafu(implicit)]
-        location: Location,
-    },
-
-    #[snafu(display("Failed to parse query body"))]
-    QueryBodyParse {
-        #[snafu(source)]
-        error: serde_json::Error,
-        #[snafu(implicit)]
-        location: Location,
-    },
-
     #[snafu(display("Missing auth token"))]
     MissingAuthToken {
         #[snafu(implicit)]
@@ -56,20 +32,6 @@ pub enum Error {
         location: Location,
     },
 
-    #[snafu(display("Invalid uuid format"))]
-    InvalidUuidFormat {
-        #[snafu(source)]
-        error: uuid::Error,
-        #[snafu(implicit)]
-        location: Location,
-    },
-
-    #[snafu(display("Missing DBT session"))]
-    MissingDbtSession {
-        #[snafu(implicit)]
-        location: Location,
-    },
-
     #[snafu(display("Invalid auth data"))]
     InvalidAuthData {
         #[snafu(implicit)]
@@ -82,14 +44,6 @@ pub enum Error {
         location: Location,
     },
 
-    #[snafu(display("Failed to parse row JSON"))]
-    RowParse {
-        #[snafu(source)]
-        error: serde_json::Error,
-        #[snafu(implicit)]
-        location: Location,
-    },
-
     #[snafu(display("UTF8 error: {error}"))]
     Utf8 {
         #[snafu(source)]
@@ -211,29 +165,14 @@ impl Error {
                     _ => (http::StatusCode::OK, SqlState::Success, error_code),
                 }
             }
-            Self::GZipDecompress { .. }
-            | Self::LoginRequestParse { .. }
-            | Self::QueryBodyParse { .. }
-            | Self::InvalidUuidFormat { .. } => {
-                // TODO: Is this need a fix? Bad request return retriable http code
-                (
-                    http::StatusCode::BAD_REQUEST,
-                    SqlState::Success,
-                    ErrorCode::Other,
-                )
-            }
             Self::MissingAuthToken { .. }
-            | Self::MissingDbtSession { .. }
             | Self::InvalidAuthData { .. }
             | Self::InvalidAuthToken { .. } => (
                 http::StatusCode::UNAUTHORIZED,
                 SqlState::Success,
                 ErrorCode::Other,
             ),
-            Self::RowParse { .. }
-            | Self::Utf8 { .. }
-            | Self::Arrow { .. }
-            | Self::NotImplemented { .. } => {
+            Self::Utf8 { .. } | Self::Arrow { .. } | Self::NotImplemented { .. } => {
                 (http::StatusCode::OK, SqlState::Success, ErrorCode::Other)
             }
         };

crates/catalog-metastore/src/error.rs

@@ -25,26 +25,6 @@ pub enum Error {
         location: Location,
     },
 
-    #[snafu(display("Volume: Validation failed. Reason: {reason}"))]
-    VolumeValidationFailed {
-        reason: String,
-        #[snafu(implicit)]
-        location: Location,
-    },
-
-    #[snafu(display("Volume: Missing credentials"))]
-    VolumeMissingCredentials {
-        #[snafu(implicit)]
-        location: Location,
-    },
-
-    #[snafu(display("Cloud provider not implemented"))]
-    CloudProviderNotImplemented {
-        provider: String,
-        #[snafu(implicit)]
-        location: Location,
-    },
-
     #[snafu(display("ObjectStore: {error}"))]
     ObjectStore {
         #[snafu(source)]
@@ -53,25 +33,6 @@ pub enum Error {
         location: Location,
     },
 
-    #[snafu(display("ObjectStore path: {error}"))]
-    ObjectStorePath {
-        #[snafu(source)]
-        error: object_store::path::Error,
-        #[snafu(implicit)]
-        location: Location,
-    },
-
-    #[snafu(display(
-        "Unable to create directory for File ObjectStore path {path}, error: {error}"
-    ))]
-    CreateDirectory {
-        path: String,
-        #[snafu(source)]
-        error: std::io::Error,
-        #[snafu(implicit)]
-        location: Location,
-    },
-
     #[snafu(display("Metastore object of type {type_name} with name {name} already exists"))]
     ObjectAlreadyExists {
         type_name: String,
@@ -80,12 +41,6 @@ pub enum Error {
         location: Location,
     },
 
-    #[snafu(display("Metastore object not found"))]
-    ObjectNotFound {
-        #[snafu(implicit)]
-        location: Location,
-    },
-
     #[snafu(display("Volume {volume} already exists"))]
     VolumeAlreadyExists {
         volume: String,