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

gen/genbzl: general improvements #76550

Merged
merged 1 commit into from
Feb 15, 2022

Conversation

ajwerner
Copy link
Contributor

This change does a few things:

  • It reworks the queries in terms of eachother in-memory. This is better than
    the previous iteration whereby it'd generate the results and then rely on
    the output of that query. Instead, we just build up bigger query expressions
    and pass them to bazel using the --query_file flag.
  • It avoids exploring the pkg/ui directory (and the pkg/gen directory) because
    those can cause problems. The pkg/ui directory ends up bringing in npm,
    which hurts.
  • It stops rewriting the files before executing the queries. It no longer
    needs to rewrite them up front because they aren't referenced by later
    queries.
  • It removes the excluded target which was problematic because those files
    weren't properly visible.

Fixes #76521
Fixes #76503

Release note: None

@ajwerner ajwerner requested a review from a team as a code owner February 15, 2022 00:46
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@ajwerner ajwerner force-pushed the ajwerner/fix-up-genbzl branch from 09a4221 to b3bf24f Compare February 15, 2022 01:55
This change does a few things:

 * It reworks the queries in terms of eachother in-memory. This is better than
   the previous iteration whereby it'd generate the results and then rely on
   the output of that query. Instead, we just build up bigger query expressions
   and pass them to bazel using the --query_file flag.
 * It avoids exploring the pkg/ui directory (and the pkg/gen directory) because
   those can cause problems. The pkg/ui directory ends up bringing in npm,
   which hurts.
 * It stops rewriting the files before executing the queries. It no longer
   needs to rewrite them up front because they aren't referenced by later
   queries.
 * It removes the excluded target which was problematic because those files
   weren't properly visible.

Fixes cockroachdb#76521
Fixes cockroachdb#76503

Release note: None
@ajwerner ajwerner force-pushed the ajwerner/fix-up-genbzl branch from b3bf24f to 80b48cf Compare February 15, 2022 17:03
@ajwerner
Copy link
Contributor Author

TFTR!

bors r+

@craig craig bot merged commit 3cb7eb0 into cockroachdb:master Feb 15, 2022
@craig
Copy link
Contributor

craig bot commented Feb 15, 2022

Build succeeded:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants