mergeCallNode does not collapse matching sibling stacks
See original GitHub issueI 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:
- Created 5 years ago
- Comments:6 (6 by maintainers)
Top 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 >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
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.
When we count samples, we loop from the sample table, not the stack table, so I think we won’t fall into this trap.