VSB-TUO/Health report license summary

[Paurikova2]

Phases	MP	MM	MB	MR	JM	Total
ETA	0	0	0	0	0	0
Developing	1	0	0	0	0	0
Review	0	0	0	0	0	0
Total	-	-	-	-	-	0
ETA est.						0
ETA cust.	-	-	-	-	-	0

Problem description

Since the database had too much data, it made too many calls and got stuck in a never-ending loop.

[Copilot]

Pull Request Overview

This PR optimizes the LicenseCheck health check by implementing batch database queries instead of individual N+1 queries, reducing the number of database calls from one per item to a single batch query. This significantly improves performance for large repositories.

Key changes:

• Added batch query method findByBitstreamUUIDs() to fetch all license mappings in a single database call
• Refactored LicenseCheck to collect bitstream UUIDs first, then batch fetch all mappings, then process results in memory
• Removed unused AWS SDK dependency (com.amazonaws.util.CollectionUtils)

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
LicenseCheck.java	Refactored to use batch queries and process results in memory; removed AWS dependency
ClarinLicenseResourceMappingService.java	Added batch query method signature
ClarinLicenseResourceMappingServiceImpl.java	Implemented batch query service method with duplicate import
ClarinLicenseResourceMappingDAO.java	Added batch query DAO interface method
ClarinLicenseResourceMappingDAOImpl.java	Implemented batch query with IN clause for multiple UUIDs

Comments suppressed due to low confidence (1)

dspace-api/src/main/java/org/dspace/health/LicenseCheck.java:46

• The contents of this container are never accessed.

        Map<UUID, Item> bitstreamToItem = new HashMap<>();

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.