Improve displayed error by using DataFusionError's Display trait (#1370)

* Use  instead of  for

* update test

Author: abey79

python/tests/test_indexing.py

@@ -43,7 +43,7 @@ def test_err(df):
     with pytest.raises(Exception) as e_info:
         df["c"]
 
-    for e in ["SchemaError", "FieldNotFound", 'name: "c"']:
+    for e in ["Schema error", "No field named c"]:
         assert e in e_info.value.args[0]
 
     with pytest.raises(Exception) as e_info:

python/tests/test_udf.py

@@ -207,4 +207,4 @@ def non_nullable_abs(input_col):
 
     with pytest.raises(Exception) as e_info:
         _results = df_result.collect()
-    assert "InvalidArgumentError" in str(e_info)
+    assert "Invalid argument error" in str(e_info)

src/errors.rs

@@ -39,7 +39,7 @@ pub enum PyDataFusionError {
 impl fmt::Display for PyDataFusionError {
     fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
         match self {
-            PyDataFusionError::ExecutionError(e) => write!(f, "DataFusion error: {e:?}"),
+            PyDataFusionError::ExecutionError(e) => write!(f, "DataFusion error: {e}"),
             PyDataFusionError::ArrowError(e) => write!(f, "Arrow error: {e:?}"),
             PyDataFusionError::PythonError(e) => write!(f, "Python error {e:?}"),
             PyDataFusionError::Common(e) => write!(f, "{e}"),