Skip to content

Commit

Permalink
Update flameshow and remove quirk for a bug in older version
Browse files Browse the repository at this point in the history
  • Loading branch information
azat committed Nov 29, 2023
1 parent 6b6ee84 commit 835218f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
2 changes: 1 addition & 1 deletion contrib/flameshow/pyoxidizer.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -249,7 +249,7 @@ def make_exe():
# attributes.
exe.add_python_resources(exe.pip_install([
"protobuf",
"flameshow",
"flameshow==1.1.1",
]))

# Invoke `pip install` using a requirements file and add the collected resources
Expand Down
3 changes: 1 addition & 2 deletions src/interpreter/clickhouse.rs
Original file line number Diff line number Diff line change
Expand Up @@ -692,8 +692,7 @@ impl ClickHouse {
WITH {} AS start_time_
SELECT
arrayStringConcat(arrayMap(
/* replaceRegexpAll() is a workaround for https://github.com/laixintao/flameshow/pull/67 */
addr -> replaceRegexpAll(demangle(addressToSymbol(addr)), ' \d+', ' '),
addr -> demangle(addressToSymbol(addr)),
arrayReverse(trace)
), ';') AS human_trace,
{} weight
Expand Down

0 comments on commit 835218f

Please sign in to comment.