-
Notifications
You must be signed in to change notification settings - Fork 4.1k
release-26.1: sql: clean up dependency graph a bit #160594
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
release-26.1: sql: clean up dependency graph a bit #160594
Conversation
Release note: None
Release note: None
Release note: None
db2af19 to
ac5a468
Compare
|
Thanks for opening a backport. Before merging, please confirm that the change does not break backwards compatibility and otherwise complies with the backport policy. Include a brief release justification in the PR description explaining why the backport is appropriate. All backports must be reviewed by the TL for the owning area. While the stricter LTS policy does not yet apply, please exercise judgment and consider gating non-critical changes behind a disabled-by-default feature flag when appropriate. |
|
It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR? 🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf. |
michae2
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@michae2 partially reviewed 16 files and all commit messages, and made 1 comment.
Reviewable status:complete! 1 of 0 LGTMs obtained (waiting on @rafiss).
Backport 3/3 commits from #160206 on behalf of @yuzefovich.
This PR contains a few commits that break a few dependencies to allow more parallelism during builds.
Informs: #79357.
Epic: None
Release note: None
Release justification: no risk build speed up.