waynr commented on code in PR #7160:
URL: https://github.com/apache/arrow-rs/pull/7160#discussion_r1964910109


##########
object_store/src/extensions.rs:
##########
@@ -0,0 +1,257 @@
+use std::{
+    any::{Any, TypeId},
+    collections::HashMap,
+    sync::Arc,
+};
+
+/// Trait that must be implemented by extensions.
+pub trait Extension: std::fmt::Debug + Send + Sync {

Review Comment:
   We have discussed that possibility, but it's not straightforward to do 
without non-trivial refactors in InfluxDB. And even if we did that, the tracing 
spans would not be properly parented within the hierarchy of tracing spans -- 
the best we would be able to do is get them parented at the root-span of a 
given query (as opposed to under the most recently-created descendant of that 
root span).



-- 
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]

Reply via email to