This is an automated email from the ASF dual-hosted git repository. wwbmmm pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/brpc.git
The following commit(s) were added to refs/heads/master by this push: new 00dcd13d Support max width of flame graph image (#2452) 00dcd13d is described below commit 00dcd13da4bc7410cbd64a0cec5e785117d860d5 Author: Bright Chen <chenguangmin...@foxmail.com> AuthorDate: Mon Nov 27 17:24:12 2023 +0800 Support max width of flame graph image (#2452) --- src/brpc/builtin/hotspots_service.cpp | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/brpc/builtin/hotspots_service.cpp b/src/brpc/builtin/hotspots_service.cpp index adb38d70..b70266f4 100644 --- a/src/brpc/builtin/hotspots_service.cpp +++ b/src/brpc/builtin/hotspots_service.cpp @@ -113,6 +113,9 @@ DEFINE_int32(max_profiles_kept, 32, "max profiles kept for cpu/heap/growth/contention respectively"); BRPC_VALIDATE_GFLAG(max_profiles_kept, PassValidate); +DEFINE_int32(max_flame_graph_width, 1200, "max width of flame graph image"); +BRPC_VALIDATE_GFLAG(max_flame_graph_width, PositiveInteger); + static const char* const PPROF_FILENAME = "pprof.pl"; static int DEFAULT_PROFILING_SECONDS = 10; static size_t CONCURRENT_PROFILING_LIMIT = 256; @@ -495,7 +498,8 @@ static void DisplayResult(Controller* cntl, if (display_type == DisplayType::kFlameGraph) { // For flamegraph, we don't care about pprof error msg, // which will cause confusing messages in the final result. - cmd_builder << " 2>/dev/null " << " | " << "perl " << flamegraph_tool; + cmd_builder << " 2>/dev/null | perl " << flamegraph_tool << " --width " + << (FLAGS_max_flame_graph_width > 0 ? FLAGS_max_flame_graph_width : 1200); } cmd_builder << " 2>&1 "; #elif defined(OS_MACOSX) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org For additional commands, e-mail: dev-h...@brpc.apache.org