coderfender commented on code in PR #21030:
URL: https://github.com/apache/datafusion/pull/21030#discussion_r3126778508
##########
datafusion/ffi/src/util.rs:
##########
@@ -17,59 +17,63 @@
use std::sync::Arc;
-use abi_stable::std_types::{RResult, RString, RVec};
use arrow::datatypes::{DataType, Field};
use arrow::ffi::FFI_ArrowSchema;
use arrow_schema::FieldRef;
+use stabby::string::String as SString;
+use stabby::vec::Vec as SVec;
use crate::arrow_wrappers::WrappedSchema;
+// Re-export for convenience
+pub use crate::ffi_option::{FFI_Option, FFI_Result};
+
/// Convenience type for results passed through the FFI boundary. Since the
/// `DataFusionError` enum is complex and little value is gained from creating
/// a FFI safe variant of it, we convert errors to strings when passing results
-/// back. These are converted back and forth using the `df_result`, `rresult`,
-/// and `rresult_return` macros.
-pub type FFIResult<T> = RResult<T, RString>;
+/// back. These are converted back and forth using the `df_result`, `sresult`,
+/// and `sresult_return` macros.
+pub type FFIResult<T> = FFI_Result<T, SString>;
Review Comment:
Absolutely! I made changes to have one result used and exposed outside
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
