marcel/familienarchiv
1
0
Fork 0
Code Issues 105 Pull Requests 2 Actions Packages Projects Releases Wiki Activity

fix(transcription): eliminate N+1 lazy load in propagation listener

Browse Source 
Switch from findByMentionedPersons_PersonId (derived query, returns blocks with
LAZY mentionedPersons) to findByPersonIdWithMentionsFetched (JOIN FETCH, loads
full collections in one round-trip). 200-block propagation: from 201 queries to 2.
Add @Transactional comment documenting join-transaction semantics.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Marcel
2026-04-28 22:15:38 +02:00
parent 1f3f879f9c
commit c7958681f5
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
backend/src/main/java/org/raddatz/familienarchiv/service/PersonMentionPropagationListener.java
View File

@@ -35,10 +35,10 @@ public class PersonMentionPropagationListener {
private final TranscriptionBlockRepository blockRepository; private final TranscriptionBlockRepository blockRepository;
@EventListener @EventListener
@Transactional @Transactional // Joins publisher's transaction — async switch requires @TransactionalEventListener(AFTER_COMMIT)
public void onPersonDisplayNameChanged(PersonDisplayNameChangedEvent event) { public void onPersonDisplayNameChanged(PersonDisplayNameChangedEvent event) {
List<TranscriptionBlock> blocks = List<TranscriptionBlock> blocks =
blockRepository.findByMentionedPersons_PersonId(event.personId()); blockRepository.findByPersonIdWithMentionsFetched(event.personId());
if (blocks.isEmpty()) { if (blocks.isEmpty()) {
return; return;
} }