Merge pull request #828 from folio-org/release_6.1.9

Release 6.1.9

NEWS.md

@@ -1,3 +1,6 @@
+## 6.1.9 2024-10-04
+  * Fixed problem with concurrentJobsGlobal not actually reading from environment variable
+
 ## 6.1.8 2024-09-09
   * ERM-3321 Description can be too long for index, causing mod-agreements error when updating from 6.0.x to later versions or failure on saving agreements with a long description
     * Swap basic GIN indices for GIN indices with trigram operator where they're not already set.

service/gradle.properties

@@ -5,7 +5,7 @@ gorm.version=7.3.3
 
 # Application
 appName=mod-agreements
-appVersion=6.1.8
+appVersion=6.1.9
 dockerTagSuffix=
 dockerRepo=folioci
 

service/grails-app/services/org/olf/general/jobs/JobRunnerService.groovy

@@ -76,11 +76,19 @@ order by pj.dateCreated
   @PostConstruct
   void init() {
     // Set up the Executor
-    if ( grailsApplication.config.getProperty('concurrentJobsGlobal') instanceof Integer && grailsApplication.config.getProperty('concurrentJobsGlobal') > 0 ) {
-      CONCURRENT_JOBS_GLOBAL = grailsApplication.config.getProperty('concurrentJobsGlobal');
+    try {
+      def concurrentJobsGlobalConfig = grailsApplication.config.getProperty('concurrentJobsGlobal', int);
+      if (concurrentJobsGlobalConfig > 0) {
+        CONCURRENT_JOBS_GLOBAL = concurrentJobsGlobalConfig;
+      }
+    } catch (Exception e) {
+      log.error("Failed to read concurrentJobsGlobal from config: ${e}")
     }
+
+    log.info("Configured jobConcurrency: ${CONCURRENT_JOBS_GLOBAL}")
+		// Base the number of small jobs executable on the limit imposed on the default runner.
 
-    // SO: This is not ideal. We don't want to limit jobs globally to 1 ideally. It should be 
+    // SO: This is not ideal. We don't want to limit jobs globally to 1 ideally. It should be
     // 1 per tenant, but that will involve implementing custom handling for the queue and executor.
     // While we only have 1 tenant, this will suffice.
     executorSvc = new ThreadPoolExecutor(