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

HCK-8816: queries optimization #120

Merged
merged 16 commits into from
Dec 5, 2024

Conversation

chulanovskyi-bs
Copy link
Contributor

@chulanovskyi-bs chulanovskyi-bs commented Nov 21, 2024

Sub-taskHCK-8816 [SQLServer] RE from instance: Filter queries based on user selection

Technical details

Comparison metrics between WHERE and WITH approaches

WHERE - https://www.brentozar.com/pastetheplan/?id=S1HElHm7yx

SELECT ...
Cached plan size	312 KB
Estimated Operator Cost	0 (0%)
Estimated Subtree Cost	0.110886
Estimated Number of Rows	7.83363

WITH - https://www.brentozar.com/pastetheplan/?id=Hkon1BX7Jx

WITH ...
Cached plan size	288 KB
Estimated Operator Cost	0 (0%)
Estimated Subtree Cost	0.155635
Estimated Number of Rows	7.83363

@chulanovskyi-bs chulanovskyi-bs changed the title HCK-8816: chores HCK-8816: queries optimization Nov 26, 2024
reverse_engineering/api.js Outdated Show resolved Hide resolved
reverse_engineering/api.js Outdated Show resolved Hide resolved
reverse_engineering/api.js Outdated Show resolved Hide resolved
@chulanovskyi-bs
Copy link
Contributor Author

@thomas-jakemeyn @bigorn0 Finalized the PR (at last 😅). The tests are passing, please check and proceed with the merge if ok

@thomas-jakemeyn thomas-jakemeyn enabled auto-merge (squash) December 5, 2024 09:37
@chulanovskyi-bs chulanovskyi-bs enabled auto-merge (squash) December 5, 2024 12:54
Copy link

sonarqubecloud bot commented Dec 5, 2024

@chulanovskyi-bs chulanovskyi-bs enabled auto-merge (squash) December 5, 2024 16:20
@chulanovskyi-bs chulanovskyi-bs merged commit b5df71e into develop Dec 5, 2024
10 checks passed
@chulanovskyi-bs chulanovskyi-bs deleted the feat/HCK-8816-optimize-queries branch December 5, 2024 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants