Improve 'updateLocations' Task for Database Efficiency

[ljdelight]

Modified the 'updateLocations' task to enhance database efficiency. The recurring task now uses a transaction per iteration instead of one for the entire task, eliminating manual commits. Cache removals are also now done with each transaction, avoiding extra database lookups.

The main point here is to reduce contention. I don't expect much change to the overall speed of the scheduled task. Validation is to check the 'finished run' time (ofc check for exceptions as well):

• docker logs maproulette-api | grep "Task 'Updating locations': Finished run"
• or journalctl -u docker --since "today" CONTAINER_NAME=maproulette-api | grep "Task 'Updating locations': Finished run"

Metrics Before change:

• Staging is consistent ~45s before the change. Prod ranges to around 50s to 90s.

[sonarqubecloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[ljdelight]

This works fine.

13:24:14.271 INFO [][application-akka.actor.default-dispatcher-35][org.maproulette.jobs.SchedulerActor] - Scheduled Task 'Updating locations': Finished run. Time spent: 46665ms
15:24:11.920 INFO [][application-akka.actor.default-dispatcher-22][org.maproulette.jobs.SchedulerActor] - Scheduled Task 'Updating locations': Finished run. Time spent: 44313ms
17:24:13.945 INFO [][application-akka.actor.default-dispatcher-5][org.maproulette.jobs.SchedulerActor] - Scheduled Task 'Updating locations': Finished run. Time spent: 46339ms