Skip to content

Commit 88a15ee

Browse files
committed
move py_err_to_datafusion_err to errors.rs
1 parent 010b4c6 commit 88a15ee

File tree

2 files changed

+8
-6
lines changed

2 files changed

+8
-6
lines changed

src/context.rs

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,9 @@ use pyo3::prelude::*;
3434
use crate::catalog::{PyCatalog, PyTable};
3535
use crate::dataframe::PyDataFrame;
3636
use crate::dataset::Dataset;
37-
use crate::errors::{py_datafusion_err, PyDataFusionError, PyDataFusionResult};
37+
use crate::errors::{
38+
py_datafusion_err, py_err_to_datafusion_err, PyDataFusionError, PyDataFusionResult,
39+
};
3840
use crate::expr::sort_expr::PySortExpr;
3941
use crate::physical_plan::PyExecutionPlan;
4042
use crate::record_batch::PyRecordBatchStream;
@@ -1438,11 +1440,6 @@ pub fn parse_file_compression_type(
14381440
})
14391441
}
14401442

1441-
/// Convert a PyErr to a DataFusionError
1442-
pub fn py_err_to_datafusion_err(err: PyErr) -> datafusion::error::DataFusionError {
1443-
datafusion::error::DataFusionError::Execution(format!("Python error: {}", err))
1444-
}
1445-
14461443
impl From<PySessionContext> for SessionContext {
14471444
fn from(ctx: PySessionContext) -> SessionContext {
14481445
ctx.ctx

src/errors.rs

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -95,3 +95,8 @@ pub fn py_unsupported_variant_err(e: impl Debug) -> PyErr {
9595
pub fn to_datafusion_err(e: impl Debug) -> InnerDataFusionError {
9696
InnerDataFusionError::Execution(format!("{e:?}"))
9797
}
98+
99+
/// Convert a PyErr to a DataFusionError
100+
pub fn py_err_to_datafusion_err(err: PyErr) -> InnerDataFusionError {
101+
InnerDataFusionError::Execution(format!("Python error: {}", err))
102+
}

0 commit comments

Comments
 (0)