Use thiserror to implement the From trait for DFSqlLogicTestError (apache#6924)

Author: jonahgao

datafusion/core/tests/sqllogictests/src/engines/datafusion/error.rs

@@ -28,45 +28,21 @@ pub type Result<T, E = DFSqlLogicTestError> = std::result::Result<T, E>;
 pub enum DFSqlLogicTestError {
     /// Error from sqllogictest-rs
     #[error("SqlLogicTest error(from sqllogictest-rs crate): {0}")]
-    SqlLogicTest(TestError),
+    SqlLogicTest(#[from] TestError),
     /// Error from datafusion
     #[error("DataFusion error: {0}")]
-    DataFusion(DataFusionError),
+    DataFusion(#[from] DataFusionError),
     /// Error returned when SQL is syntactically incorrect.
     #[error("SQL Parser error: {0}")]
-    Sql(ParserError),
+    Sql(#[from] ParserError),
     /// Error from arrow-rs
     #[error("Arrow error: {0}")]
-    Arrow(ArrowError),
+    Arrow(#[from] ArrowError),
     /// Generic error
     #[error("Other Error: {0}")]
     Other(String),
 }
 
-impl From<TestError> for DFSqlLogicTestError {
-    fn from(value: TestError) -> Self {
-        DFSqlLogicTestError::SqlLogicTest(value)
-    }
-}
-
-impl From<DataFusionError> for DFSqlLogicTestError {
-    fn from(value: DataFusionError) -> Self {
-        DFSqlLogicTestError::DataFusion(value)
-    }
-}
-
-impl From<ParserError> for DFSqlLogicTestError {
-    fn from(value: ParserError) -> Self {
-        DFSqlLogicTestError::Sql(value)
-    }
-}
-
-impl From<ArrowError> for DFSqlLogicTestError {
-    fn from(value: ArrowError) -> Self {
-        DFSqlLogicTestError::Arrow(value)
-    }
-}
-
 impl From<String> for DFSqlLogicTestError {
     fn from(value: String) -> Self {
         DFSqlLogicTestError::Other(value)