The GitHub Actions job "Required Checks" on 
texera.git/feat/7743-warehouse-owner-fields has failed.
Run started by GitHub user mengw15 (triggered by mengw15).

Head commit for run:
efa90521aafb5123976460c35c0aa3bf8b92ef6f / mengw15 
<[email protected]>
refactor(amber): resolve the warehouse owner in the listing query

Review feedback on the owner fields:

The listing fetched warehouses and then looked their owners up in a
second query. Join instead, so one round trip carries both. `create`
went to the database at all, though the caller is the owner it is
resolving and SessionUser already holds their display info -- it now
reads from there.

What remains of the resolution is turning an unset name or avatar into
null, so that is all `ownerOf` does. The `(null, null)` fallback a
reviewer asked to name is gone rather than named: with the left join
supplying nulls directly and `create` reading the session, nothing is
left that has to invent an absent owner.

Report URL: https://github.com/apache/texera/actions/runs/32226138442

With regards,
GitHub Actions via GitBox

Reply via email to