Refactor queries_grouped by hash and 2 additional optimization scripts from PR #471#517

Merged

danieldeleo merged 11 commits into

GoogleCloudPlatform:masterfrom

Andres-Ayala1:optimization-additions

Feb 17, 2026

Andres-Ayala1 commented Dec 22, 2025

Contributor

Ref PR: #471

Added the requested edits from this PR:

Consolidated duration percentile columns into queries_grouped_by_hash_org.sql and queries_grouped_by_hash_project.sql
- Remove separate *_duration.sql files
Removed newline at end of bi_engine_mode_duration.sql

afleisc and others added 5 commits

January 14, 2025 13:59


          refactor optimization scripts add 4 additional scripts

3a661ab


          chore: remove newline at end of file

facc0ee


          feat: consolidate duration percentiles into query hash org and proj l…

26408f5

…evel scripts


          chore: remove old duration scripts and update README.md

31788ba


          chore: remove newline in queries by hash org

eb1968e

Andres-Ayala1 mentioned this pull request

Refactor queries_grouped_by_hash and add 4 additional Optimization Scripts #471

Open

Andres-Ayala1 commented Jan 28, 2026

Contributor Author

@danieldeleo Seems bot did not auto-assign, please review and provide any additional changes needed when you have time :)

afleisc self-requested a review

February 9, 2026 16:03

afleisc approved these changes

View reviewed changes

afleisc commented Feb 9, 2026

Contributor

@danieldeleo I'm good with these changes if you'd like to take a final pass

danieldeleo reviewed

View reviewed changes

scripts/optimization/bi_engine_disabled_reasons.sql Outdated

danieldeleo reviewed

View reviewed changes

scripts/optimization/bi_engine_mode_duration.sql Outdated

danieldeleo reviewed

View reviewed changes

scripts/optimization/queries_grouped_by_hash_project.sql Outdated

danieldeleo reviewed

View reviewed changes

scripts/optimization/README.md Outdated

danieldeleo reviewed

View reviewed changes

scripts/optimization/README.md Outdated

danieldeleo requested changes

View reviewed changes

danieldeleo left a comment

Collaborator

Left a few comments for changes, thanks!

Andres-Ayala1 added 5 commits

February 11, 2026 14:58


          chore: update license year and descriptions

0aaa621


          fix: remove ORDER BY from table creation script + add readme example

7c44804


          fix: remove ORDER BY and limit from queries_grouped_by_hash_project +…

0f6f8e6

… add README example queries


          chore: move bi engine tables into Query analysis section + add exampl…

055e86b

…e queries


          chore: add description for bi_engine_disabled_reasons

a30f626

Andres-Ayala1 requested a review from danieldeleo

February 11, 2026 21:46

Andres-Ayala1 commented Feb 11, 2026

Contributor Author

@afleisc @danieldeleo I have

updated the licenses
Removed ordering and limits from table creation scripts
added examples for new query analysis scripts
moved bi engine table scripts to query analysis section in README

danieldeleo requested changes

View reviewed changes

scripts/optimization/queries_grouped_by_script.sql Outdated


          fix: license years on existing scripts

3904a9c

Andres-Ayala1 requested a review from danieldeleo

February 17, 2026 19:48

danieldeleo approved these changes

View reviewed changes

danieldeleo left a comment

Collaborator

Thanks!

danieldeleo merged commit 56aa999 into GoogleCloudPlatform:master

88 checks passed

Hackerpilot pushed a commit to Hackerpilot/bigquery-utils that referenced this pull request


          Refactor queries_grouped by hash and 2 additional optimization script…

2b48e80

…s from PR GoogleCloudPlatform#471 (GoogleCloudPlatform#517)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet