More instrument adapter (#5588)

* instrument adapter publish, execute
MystenLabs · Oct 27, 2022 · cce8063 · cce8063
1 parent bae3c85
commit cce8063
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 0 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/crates/sui-adapter/Cargo.toml b/crates/sui-adapter/Cargo.toml
@@ -13,6 +13,7 @@ bcs = "0.1.4"
 leb128 = "0.2.5"
 once_cell = "1.14.0"
 linked-hash-map = "0.5.6"
+tracing = "0.1.36"
 
 move-binary-format.workspace = true
 move-core-types.workspace = true

diff --git a/crates/sui-adapter/src/adapter.rs b/crates/sui-adapter/src/adapter.rs
@@ -48,6 +48,7 @@ use sui_verifier::{
     entry_points_verifier::{is_tx_context, RESOLVED_ASCII_STR, RESOLVED_UTF8_STR},
     verifier, INIT_FN_NAME,
 };
+use tracing::instrument;
 
 use crate::bytecode_rewriter::ModuleHandleRewriter;
 
@@ -91,6 +92,7 @@ pub fn new_session<
 /// otherwise we return Ok(Ok).
 /// TODO: Do we really need the two layers?
 #[allow(clippy::too_many_arguments)]
+#[instrument(name = "adapter_execute", level = "trace", skip_all)]
 pub fn execute<
     E: Debug,
     S: ResourceResolver<Error = E>
@@ -293,6 +295,7 @@ fn execute_internal<
     Ok(())
 }
 
+#[instrument(name = "adapter_publish", level = "trace", skip_all)]
 pub fn publish<
     E: Debug,
     S: ResourceResolver<Error = E>