test(blockifier): use the jemalloc allocator for benchmark

[ArniStarkware]

No description provided.

[reviewable-StarkWare]

This change is 

[ArniStarkware]

• feat(starknet_sequencer_node): hide jemalloc under feature in node main #3898
• test(blockifier): use the jemalloc allocator for benchmark #3899 👈 (View in Graphite)
• feat(starknet_sequencer_node): set jemalloc as global allocator #3897
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[github-actions]

Artifacts upload workflows:

• Started at 2025-02-03T08:52:10Z
• Started at 2025-02-03T09:02:16Z
• Started at 2025-02-03T09:13:50Z
• Started at 2025-02-03T09:16:04Z
• Started at 2025-02-03T13:37:49Z
• Started at 2025-02-03T13:48:00Z
• Started at 2025-02-03T14:26:36Z

[alonh5]

Reviewed 2 of 3 files at r1, all commit messages.
Reviewable status: 2 of 3 files reviewed, 1 unresolved discussion (waiting on @ArniStarkware)

---

crates/blockifier/bench/blockifier_bench.rs line 17 at r1 (raw file):

use criterion::{criterion_group, criterion_main, Criterion};

// Note, it might be interesting to run this benchmark with and without the Jemalloc allocator.

Turn this into a TODO to consider.
Also, we're not currently running the bench in the CI right? We should probably add it right?

Code quote:

// Note, it might be interesting to run this benchmark with and without the Jemalloc allocator.

[ArniStarkware]

Reviewable status: 2 of 3 files reviewed, 1 unresolved discussion (waiting on @alonh5)

---

crates/blockifier/bench/blockifier_bench.rs line 17 at r1 (raw file):

Previously, alonh5 (Alon Haramati) wrote…

Turn this into a TODO to consider.
Also, we're not currently running the bench in the CI right? We should probably add it right?

Done.

About CI:

1. We don't run this bench in CI.
2. "Should we?" The same question can be asked about other benches in this repo. For example, the gateway bench.
3. "If we want to do it - how should we do it?" We can do something similar to how they do it here: crates/committer_cli/benches/bench_split_and_prepare_post.sh.

[alonh5]

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @ArniStarkware)

---

crates/blockifier/bench/blockifier_bench.rs line 17 at r1 (raw file):

Previously, ArniStarkware (Arnon Hod) wrote…

Done.

About CI:

1. We don't run this bench in CI.
2. "Should we?" The same question can be asked about other benches in this repo. For example, the gateway bench.
3. "If we want to do it - how should we do it?" We can do something similar to how they do it here: crates/committer_cli/benches/bench_split_and_prepare_post.sh.

Okay I'll raise this question in the next sync

[alonh5]

Reviewed 68 of 68 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @ArniStarkware)