question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

mergeCallNode does not collapse matching sibling stacks

See original GitHub issue

I was reading mergeCallNode and I think I found something that could be argued to be a bug.

If your stack tree looks like this:

 - (root)
   - frame-to-merge
     - frame-to-collapse
   - frame-to-collapse

and you merge the call node frame-to-merge into its caller, then you’ll end up with the following tree in your stackTable:

 - (root)
   - frame-to-collapse
   - frame-to-collapse

The two sibling nodes will end up being collapsed in the call node table, once the call node table is computed, but they aren’t collapsed in the stackTable. This is currently not observable in the UI but it might become observable in the future.

Issue Analytics

  • State:closed
  • Created 5 years ago
  • Comments:6 (6 by maintainers)

github_iconTop GitHub Comments

1reaction
mstangecommented, Sep 4, 2021

I’ve read this again and still agree with the conclusion: Not having sibling stacks with the same frame is not a property we rely on, and if we had such a guarantee, it wouldn’t unlock any simplifications. So everything’s fine the way it is. Closing.

0reactions
julienwcommented, Jun 14, 2018

When we count samples, we loop from the sample table, not the stack table, so I think we won’t fall into this trap.

Read more comments on GitHub >

github_iconTop Results From Across the Web

Selecting siblings between two nodes using XPath
Use the Kayessian method of node-set intersection · The intersection of two node-sets $ns1 and $ns2 is · If we have the following...
Read more >
Pipeline aggregations | Elasticsearch Guide [master] | Elastic
Sibling : Pipeline aggregations that are provided with the output of a ... into a bucket do not contain a required field; There...
Read more >
JProfiler Help - Call tree analyses - ej-technologies
Below that node, stacks from different depths are merged. The number of merged stacks is shown in the tool tip as well. The...
Read more >
Just-in-Time Mode - Tailwind CSS
You can even stack them like sm:hover:active:disabled:opacity-75 . Never configure your variants again. Generate arbitrary styles without writing custom CSS.
Read more >
nltk.tree - NLTK
:param collapsePOS: 'False' (default) will not collapse the parent of leaf ... of trees. stack = [(None, [])] # list of (node, children)...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found