The GitHub Actions job "Build Health" on 
grails-core.git/fix/multi-tenant-get-by-id has succeeded.
Run started by GitHub user jdaugherty (triggered by jdaugherty).

Head commit for run:
34c879e803f516497809555d8cee157d3fbdbbde / James Daugherty 
<[email protected]>
Return held instances of the current tenant from lookups by id

A lookup by id of a multi-tenant entity now returns the instance the
session already holds when it belongs to the current tenant, including
one saved but not yet flushed, and queries the others without flushing
the session. GORM for MongoDB does not flush before a query inside a
transaction, so an instance saved earlier in that transaction was not
found, and outside one every lookup by id flushed the whole session.
load returns the held instance again instead of a proxy.

Tenants.withoutId is detected from the tenant bound to the thread, so a
tenant resolver that resolves DEFAULT no longer lifts the restriction
for lookups by id while queries still apply it. getAll without ids
returns an empty list without needing a current tenant.

Report URL: https://github.com/apache/grails-core/actions/runs/37935393627

With regards,
GitHub Actions via GitBox

Reply via email to