Skip to content

Commit 621b31d

Browse files
authored
apollo_class_manager: add log when finished compilation (#10809)
1 parent 75a6137 commit 621b31d

File tree

1 file changed

+12
-1
lines changed

1 file changed

+12
-1
lines changed

crates/apollo_class_manager/src/class_manager.rs

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
use std::time::Instant;
2+
13
use apollo_class_manager_config::config::{ClassManagerConfig, FsClassManagerConfig};
24
use apollo_class_manager_types::{
35
ClassHashes,
@@ -15,7 +17,7 @@ use apollo_compile_to_casm_types::{
1517
use apollo_infra::component_definitions::{default_component_start_fn, ComponentStarter};
1618
use async_trait::async_trait;
1719
use starknet_api::state::{SierraContractClass, CONTRACT_CLASS_VERSION};
18-
use tracing::instrument;
20+
use tracing::{debug, instrument};
1921

2022
use crate::class_storage::{CachedClassStorage, ClassStorage, FsClassStorage};
2123
use crate::metrics::register_metrics;
@@ -60,6 +62,7 @@ where
6062
return Ok(ClassHashes { class_hash, executable_class_hash_v2 });
6163
}
6264

65+
let compilation_start_time = Instant::now();
6366
let (raw_executable_class, executable_class_hash_v2) =
6467
self.compiler.compile(class.clone()).await.map_err(|err| match err {
6568
SierraCompilerClientError::SierraCompilerError(error) => {
@@ -69,6 +72,14 @@ where
6972
ClassManagerError::Client(error.to_string())
7073
}
7174
})?;
75+
debug!(
76+
%class_hash,
77+
compiled_class_hash = %executable_class_hash_v2,
78+
compilation_elapsed_ms = compilation_start_time.elapsed().as_millis(),
79+
class_size_bytes = class.size().map(|size| size.to_string()).unwrap_or("Failed to get class size".to_owned()),
80+
casm_size_bytes = raw_executable_class.size().map(|size| size.to_string()).unwrap_or("Failed to get casm size".to_owned()),
81+
"Finished compiling class."
82+
);
7283

7384
self.validate_class_length(&raw_executable_class)?;
7485
Self::validate_class_version(&sierra_class)?;

0 commit comments

Comments
 (0)