Zoey Greer comments

Results 42 comments of


                                            Zoey Greer

Remove transaction surrounding `createAPIKey` call

* **#6221** 👈 * **#6192** * **#6180** * **#6045** * **#6043** : 1 other dependent PR ([#6044](https://github.com/buildbuddy-io/buildbuddy/pull/6044) ) * **#6014** * **#6012** * **#6010** * **#6003** * **#5865** : 1...

Use `JOIN` for quota fetch

> [!WARNING] > This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite....

We no longer need to wrap getUser in a transaction

* **#6221** * **#6192** * **#6180** 👈 * **#6045** * **#6043** : 1 other dependent PR ([#6044](https://github.com/buildbuddy-io/buildbuddy/pull/6044) ) * **#6014** * **#6012** * **#6010** * **#6003** * **#5865** : 1...

Get user's group memberships in the same query as the user

* **#6221** * **#6192** * **#6180** * **#6045** 👈 * **#6043** : 1 other dependent PR ([#6044](https://github.com/buildbuddy-io/buildbuddy/pull/6044) ) * **#6014** * **#6012** * **#6010** * **#6003** * **#5865** : 1...

Remove unnecessary select in delete invocation with perms

> [!WARNING] > This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite....

Remove unnecessary select statement when updating targets

* **#6043** : 2 dependent PRs ([#6044](https://github.com/buildbuddy-io/buildbuddy/pull/6044) , [#6045](https://github.com/buildbuddy-io/buildbuddy/pull/6045) ) * **#6014** 👈 * **#6012** * **#6010** * **#6003** * **#5865** : 1 other dependent PR ([#5875](https://github.com/buildbuddy-io/buildbuddy/pull/5875) ) * **#6002**...

Uses a single query instead of two queries to retrieve a user by subid

* **#6043** : 2 dependent PRs ([#6044](https://github.com/buildbuddy-io/buildbuddy/pull/6044) , [#6045](https://github.com/buildbuddy-io/buildbuddy/pull/6045) ) * **#6014** * **#6012** 👈 * **#6010** * **#6003** * **#5865** : 1 other dependent PR ([#5875](https://github.com/buildbuddy-io/buildbuddy/pull/5875) ) * **#6002**...

remove transactions surrounding single queries

Current dependencies on/for this PR: * **#5865** : 2 dependent PRs ([#5875](https://github.com/buildbuddy-io/buildbuddy/pull/5875) , [#6002](https://github.com/buildbuddy-io/buildbuddy/pull/6002) ) * **#5864** 👈 * **#5863** * **#5862** * `master` This [stack of pull requests](https://stacking.dev/?utm_source=stack-comment) is...

shrink transactions with initial non-locking reads and single terminating writes

Current dependencies on/for this PR: * **#5865** : 2 dependent PRs ([#5875](https://github.com/buildbuddy-io/buildbuddy/pull/5875) , [#6002](https://github.com/buildbuddy-io/buildbuddy/pull/6002) ) * **#5864** * **#5863** 👈 * **#5862** * `master` This [stack of pull requests](https://stacking.dev/?utm_source=stack-comment) is...

wip

Current dependencies on/for this PR: * `master` * **PR #5456** 👈 This [stack of pull requests](https://stacking.dev/?utm_source=stack-comment) is managed by [Graphite](https://app.graphite.dev/github/pr/buildbuddy-io/buildbuddy/5456?utm_source=stack-comment).