paleolimbot commented on code in PR #2:
URL: https://github.com/apache/sedona-db/pull/2#discussion_r2310610321
##########
rust/sedona-common/src/error.rs:
##########
@@ -0,0 +1,34 @@
+/// Macro to create Sedona Internal Error that avoids the misleading error
message from
+/// DataFusionError::Internal.
+#[macro_export]
+macro_rules! sedona_internal_err {
+ ($($args:expr),*) => {{
+ let msg = std::format!(
+ "SedonaDB internal error: {}{}.\nThis issue was likely caused by a
bug in SedonaDB's code. \
+ Please help us to resolve this by filing a bug report in our issue
tracker: \
+ https://github.com/apache/sedona-db/issues";,
+ std::format!($($args),*),
+ datafusion_common::DataFusionError::get_back_trace(),
Review Comment:
If we get the same linking errors in CI that we got before, you could try
removing the back trace feature (we can always add it back in later if it turns
out that we're having trouble debugging user errors)
--
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: dev-unsubscr...@sedona.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
