The GitHub Actions job "CI" on grails-core.git/feat/gorm-datastore-infra has 
failed.
Run started by GitHub user borinquenkid (triggered by borinquenkid).

Head commit for run:
0f8e6c6572dc60b07d820d11758a867e4d5794e6 / Walter Duque de Estrada 
<[email protected]>
fix: address Copilot review findings on PR #15779

- ThreadLocalSessionResolver: qualifier-bound sessions were stored in a
  single shared ConcurrentHashMap, leaking across threads; move to a
  ThreadLocal<Map> and clear it on unbind().
- AbstractDatastore.DefaultApplicationEventPublisher: dispatched every
  event to every listener regardless of declared generic type, risking
  ClassCastException for typed listeners; filter via Spring's own
  GenericApplicationListenerAdapter/ResolvableType before invoking, and
  make the listener list a CopyOnWriteArrayList.
- DatastoreUtils.bindSession: silently skipped binding when a
  SessionHolder was already present for the datastore, which could
  leave a freshly created session unbound (e.g. DataTestSetupInterceptor
  creates a new session per test method). Add the session to the
  existing holder instead, matching bindNewSession's push/pop semantics.
- Align new spec file license headers with the standard ASF header used
  elsewhere in the module.

Co-Authored-By: Claude Sonnet 5 <[email protected]>

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

With regards,
GitHub Actions via GitBox

Reply via email to