Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WorkloadViewer shows queries twice, with inverted stats (baseline <--> benchmark) #115

Closed
CrazySwimmer opened this issue Feb 27, 2022 · 6 comments
Labels
bug Something isn't working

Comments

@CrazySwimmer
Copy link

It is the exact same workload captured in two different ways.
Baseline was captured from ExtendedEventsWorkloadListener to AnalysisConsumer + WorkloadFileWriterConsumer
Benchmark was captured from FileWorkloadListener to AnalysisConsumer
The benchmark does not come from a replay.

The query marked in red appears only once because I stopped the benchmark analysis before it was finished.

image

@CrazySwimmer
Copy link
Author

I can confirm that I get the same result even after full replays, no matter what two workloads I compare.

An interesting point I noticed is that I can select either of any two queries and the "Query Details" pane will always show only one result (i.e., not inverted).
I would guess this points to a problem in how the merge is done in the "Queries" pane.

@spaghettidba
Copy link
Owner

Thank you for reporting this problem.
Would it be possible for you to upload a copy of the analysis database? You can "anonymize" the queries but blanking the "normalized_text" and "example_text" columns in the NormalizedQueries table.

@CrazySwimmer
Copy link
Author

Hi Gianluca,
Apologies for the late reply.
Where should I upload the analysis database?

@spaghettidba
Copy link
Owner

Wherever you prefer (google drive, onedrive...). If the database is small enough you can attach it to the issue by drag/drop on the comment.

@CrazySwimmer
Copy link
Author

I took the liberty to anonymize databases, hosts and logins as well.
One precision: this isn't the database from the screenshot above, but the "feature" of interest is still present.

Here's a ZIP containing the backup file:
WorkloadAnalysis_CrazySwimmer.zip

@spaghettidba
Copy link
Owner

Awesome! Thank you very much for the file. I will have a look as soon as possible.

@spaghettidba spaghettidba added the bug Something isn't working label Apr 18, 2023
spaghettidba added a commit that referenced this issue Nov 24, 2023
Fix #115 WorkloadViewer shows queries twice
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants