profiler icon indicating copy to clipboard operation
profiler copied to clipboard

Inconsistent application of multiple search terms

Open mstange opened this issue 5 years ago • 5 comments

In the search fields in our various panes, you can use commas to search for multiple strings. However, it seems that different panes have different interpretations of "searching for multiple strings":

  • If I search the call tree for addon,extension, I only get call stacks where both the string "addon" and the string "extension" is present in the stack: https://perfht.ml/2QjbZtB
  • If I search the marker chart for ipcin,display, I get markers that match just one of the terms: https://perfht.ml/2QmNlrU

Please choose either OR or AND and make it consistent.

┆Issue is synchronized with this Jira Task

mstange avatar Mar 17 '20 22:03 mstange

The multiple search for markers is quite recent.

Which way do you think would be more useful for you Markus? I think the AND behavior is more useful and that's why I did it for the calltree initially, because we can always get a OR behavior by searching twice. What do you think?

julienw avatar Mar 18 '20 10:03 julienw

I'm not sure - ideally I'd want access to both :) But if only one is available, it should really be consistent.

I think the AND behavior is more useful and that's why I did it for the calltree initially, because we can always get a OR behavior by searching twice.

I actually disagree with the "searching twice" argument. In the first profile link given above, I was actually trying to find out "how many samples are we spending in extension-related code?" Some of those samples only have the word "addon" in their stack and some only have the word "extension" in their stack, but some have both. So I can't really just add samples with "addon" + samples with "extension", because I might count the samples in the overlap twice. I guess I could add the two and then subtract samples with "addon,extension"...

mstange avatar Mar 18 '20 15:03 mstange

I see, thanks for the feedback! I think I'd implement the AND behavior for both, but we should find a better way. I think we actually had some plans some years ago but never really looked at it since then.

julienw avatar Mar 18 '20 16:03 julienw

Was recently trying to find specific markers that matched a part of a URL while ignoring the other markers and having an AND would be very useful!

dpalmeiro avatar Sep 23 '20 13:09 dpalmeiro

Can we remove the comma feature and only use a regex, then users can craft all kinds of crazy searches without inventing lots of UI to handle it. This would be similar to how searchfox works.

gregtatum avatar Oct 09 '20 16:10 gregtatum