Skip to content

Optimize getQueueEntries query performance - reduce joins from 59 to 11 #92

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
shubhangiisinghh wants to merge 3 commits into
base: main
Choose a base branch
from

Fix: Deduplicate JOIN FETCH results using LinkedHashSet

13fc8d2
Select commit
Loading
Failed to load commit list.
Open

Optimize getQueueEntries query performance - reduce joins from 59 to 11 #92

Fix: Deduplicate JOIN FETCH results using LinkedHashSet
13fc8d2
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded May 18, 2026 in 17s

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check failure on line 263 in api/src/main/java/org/openmrs/module/queue/api/dao/impl/QueueEntryDaoImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "qe.startedAt" 3 times. 

See more on https://sonarcloud.io/project/issues?id=openmrs_openmrs-module-queue&issues=AZ45ipb8iLpuLITvgZeE&open=AZ45ipb8iLpuLITvgZeE&pullRequest=92

Check failure on line 45 in api/src/main/java/org/openmrs/module/queue/api/dao/impl/QueueEntryDaoImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 29 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=openmrs_openmrs-module-queue&issues=AZ45ipb8iLpuLITvgZeF&open=AZ45ipb8iLpuLITvgZeF&pullRequest=92
View more details on SonarQubeCloud