EVA-3494 Ingest assembly in batches

src/main/java/uk/ac/ebi/eva/contigalias/ContigAliasApplication.java

@@ -22,11 +22,15 @@
 import org.springframework.boot.web.servlet.support.SpringBootServletInitializer;
 import org.springframework.hateoas.config.EnableHypermediaSupport;
 import org.springframework.retry.annotation.EnableRetry;
+import org.springframework.scheduling.annotation.EnableAsync;
 import org.springframework.scheduling.annotation.EnableScheduling;
+import org.springframework.transaction.annotation.EnableTransactionManagement;
 
 @EnableScheduling
+@EnableAsync
 @SpringBootApplication
 @EnableRetry
+@EnableTransactionManagement
 @EnableHypermediaSupport(type = EnableHypermediaSupport.HypermediaType.HAL)
 public class ContigAliasApplication extends SpringBootServletInitializer {
 

src/main/java/uk/ac/ebi/eva/contigalias/conf/ApplicationContextHolder.java

@@ -0,0 +1,21 @@
+package uk.ac.ebi.eva.contigalias.conf;
+
+import org.springframework.beans.BeansException;
+import org.springframework.context.ApplicationContext;
+import org.springframework.context.ApplicationContextAware;
+import org.springframework.stereotype.Component;
+
+@Component
+public class ApplicationContextHolder implements ApplicationContextAware {
+
+    private static ApplicationContext applicationContext;
+
+    @Override
+    public void setApplicationContext(ApplicationContext applicationContext) throws BeansException {
+        ApplicationContextHolder.applicationContext = applicationContext;
+    }
+
+    public static ApplicationContext getApplicationContext() {
+        return applicationContext;
+    }
+}

src/main/java/uk/ac/ebi/eva/contigalias/controller/admin/AdminController.java

@@ -33,8 +33,6 @@
 import java.io.IOException;
 import java.util.List;
 import java.util.Map;
-import java.util.Set;
-import java.util.stream.Collectors;
 
 @RequestMapping("/v1/admin")
 @RestController
@@ -63,6 +61,9 @@ public ResponseEntity<?> fetchAndInsertAssemblyByAccession(
                     "GCA_000001405.10") String asmAccession) throws IOException {
         try {
             handler.fetchAndInsertAssemblyByAccession(asmAccession);
+            // submit jobs for updating ena sequence name and md5 checksum for assembly
+            handler.retrieveAndInsertENASequenceNameForAssembly(asmAccession);
+            handler.retrieveAndInsertMd5ChecksumForAssembly(asmAccession);
         } catch (IllegalArgumentException e) {
             return new ResponseEntity<>(e.getMessage(), HttpStatus.BAD_REQUEST);
         }
@@ -82,17 +83,22 @@ public ResponseEntity<?> fetchAndInsertAssemblyByAccession(
                     "parallel manner.")
     @PutMapping(value = "assemblies")
     public ResponseEntity<?> fetchAndInsertAssemblyByAccession(
-            @RequestBody(required = false) @ApiParam(value = "A JSON array of INSDC or RefSeq assembly accessions. " +
+            @RequestBody @ApiParam(value = "A JSON array of INSDC or RefSeq assembly accessions. " +
                     "Eg: [\"GCA_000001405.10\",\"GCA_000001405.11\",\"GCA_000001405.12\"]") List<String> accessions) {
         if (accessions == null || accessions.size() <= 0) {
             return new ResponseEntity<>(HttpStatus.BAD_REQUEST);
         }
         Map<String, List<String>> accessionResult = handler.fetchAndInsertAssemblyByAccession(accessions);
+        // submit jobs for updating ena sequence names and md5 checksum for all successfully inserted assemblies
+        if (accessionResult.get("SUCCESS").size() > 0) {
+            handler.retrieveAndInsertENASequenceNameForAssembly(accessionResult.get("SUCCESS"));
+            handler.retrieveAndInsertMd5ChecksumForAssembly(accessionResult.get("SUCCESS"));
+        }
         return new ResponseEntity<>("Accession Processing Result : " + accessionResult, HttpStatus.MULTI_STATUS);
     }
 
     @ApiOperation(value = "Given an assembly accession, retrieve MD5 checksum for all chromosomes belonging to assembly and update")
-    @PutMapping(value = "assemblies/{accession}/md5checksum")
+    @PutMapping(value = "assemblies/md5checksum/{accession}")
     public ResponseEntity<String> retrieveAndInsertMd5ChecksumForAssembly(@PathVariable(name = "accession")
                                                                           @ApiParam(value = "INSDC or RefSeq assembly accession. Eg: " +
                                                                                   "GCA_000001405.10") String asmAccession) {
@@ -101,24 +107,60 @@ public ResponseEntity<String> retrieveAndInsertMd5ChecksumForAssembly(@PathVaria
             handler.retrieveAndInsertMd5ChecksumForAssembly(asmAccession);
             return ResponseEntity.ok("A task has been submitted for updating md5checksum for all chromosomes " +
                     "in assembly " + asmAccession + ". Depending upon the number of chromosomes present in assembly, " +
-                    "this might take some time to complete");
+                    "and other scheduled jobs, this might take some time to complete");
         } catch (AssemblyNotFoundException e) {
             return ResponseEntity.ok("Could not find assembly " + asmAccession +
                     ". Please insert the assembly first (md5checksum will be updated as part of the insertion process");
         }
     }
 
+    @ApiOperation(value = "Given a list of assembly accessions, retrieve MD5 checksum for all chromosomes belonging to all the assemblies and update")
+    @PutMapping(value = "assemblies/md5checksum")
+    public ResponseEntity<String> retrieveAndInsertMd5ChecksumForAssembly(
+            @RequestBody @ApiParam(value = "A JSON array of INSDC or RefSeq assembly accessions. " +
+                    "Eg: [\"GCA_000001405.10\",\"GCA_000001405.11\",\"GCA_000001405.12\"]") List<String> accessions) {
+        handler.retrieveAndInsertMd5ChecksumForAssembly(accessions);
+        return ResponseEntity.ok("A task has been submitted for updating md5checksum for all chromosomes " +
+                "in assemblies " + accessions + ". Depending upon the number of chromosomes present in assembly, " +
+                "and other scheduled jobs, this might take some time to complete");
+
+    }
+
+    @ApiOperation(value = "Given an assembly accession, retrieve ENA sequence name for all chromosomes belonging to assembly and update")
+    @PutMapping(value = "assemblies/ena-sequence-name/{accession}")
+    public ResponseEntity<String> retrieveAndInsertENASequenceNameForAssembly(@PathVariable(name = "accession")
+                                                                              @ApiParam(value = "INSDC or RefSeq assembly accession. " +
+                                                                                      "Eg: GCA_000001405.10") String asmAccession) {
+        try {
+            handler.getAssemblyByAccession(asmAccession);
+            handler.retrieveAndInsertENASequenceNameForAssembly(asmAccession);
+            return ResponseEntity.ok("A task has been submitted for updating ENA Sequence Name for all chromosomes " +
+                    "in assembly " + asmAccession + ". Depending upon the number of chromosomes present in assembly, " +
+                    "and other scheduled jobs, this might take some time to complete");
+        } catch (AssemblyNotFoundException e) {
+            return ResponseEntity.ok("Could not find assembly " + asmAccession +
+                    ". Please insert the assembly first (ENA sequence name will be updated as part of the insertion process");
+        }
+    }
+
+    @ApiOperation(value = "Given a list of assembly accessions, retrieve ENA sequence name for all chromosomes belonging to all the assemblies and update")
+    @PutMapping(value = "assemblies/ena-sequence-name")
+    public ResponseEntity<String> retrieveAndInsertENASequenceNameForAssembly(
+            @RequestBody @ApiParam(value = "A JSON array of INSDC or RefSeq assembly accessions. " +
+                    "Eg: [\"GCA_000001405.10\",\"GCA_000001405.11\",\"GCA_000001405.12\"]") List<String> accessions) {
+
+        handler.retrieveAndInsertENASequenceNameForAssembly(accessions);
+        return ResponseEntity.ok("A task has been submitted for updating ENA Sequence Name for all chromosomes " +
+                "in assembly " + accessions + ". Depending upon the number of chromosomes present in assembly, " +
+                "and other scheduled jobs, this might take some time to complete");
+    }
+
+
     @ApiOperation(value = "Retrieve list of assemblies for which MD5 Checksum updates are running/going-to-run ")
-    @GetMapping(value = "assemblies/md5checksum/status")
-    public ResponseEntity<String> getMD5ChecksumUpdateTaskStatus() {
-        Map<String, Set<String>> md5ChecksumUpdateTasks = handler.getMD5ChecksumUpdateTaskStatus();
-        Set<String> runningTasks = md5ChecksumUpdateTasks.get("running");
-        Set<String> scheduledTasks = md5ChecksumUpdateTasks.get("scheduled");
-        String runningTaskRes = runningTasks == null || runningTasks.isEmpty() ? "No running MD5 checksum update tasks" :
-                runningTasks.stream().collect(Collectors.joining(","));
-        String scheduledTaskRes = scheduledTasks == null || scheduledTasks.isEmpty() ? "No scheduled MD5 checksum update tasks" :
-                scheduledTasks.stream().collect(Collectors.joining(","));
-        return ResponseEntity.ok("running: " + runningTaskRes + "\nscheduled: " + scheduledTaskRes);
+    @GetMapping(value = "assemblies/scheduled-jobs")
+    public ResponseEntity<List<String>> getMD5ChecksumUpdateTaskStatus() {
+        List<String> scheduledJobStatus = handler.getScheduledJobStatus();
+        return ResponseEntity.ok(scheduledJobStatus);
     }
 
 //    This endpoint can be enabled in the future when checksums for assemblies are added to the project.

src/main/java/uk/ac/ebi/eva/contigalias/controller/admin/AdminHandler.java

@@ -19,16 +19,13 @@
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.data.web.PagedResourcesAssembler;
 import org.springframework.stereotype.Service;
-
 import uk.ac.ebi.eva.contigalias.entities.AssemblyEntity;
 import uk.ac.ebi.eva.contigalias.service.AssemblyService;
 import uk.ac.ebi.eva.contigalias.service.ChromosomeService;
 
-import java.io.IOException;
 import java.util.List;
 import java.util.Map;
 import java.util.Optional;
-import java.util.Set;
 
 @Service
 public class AdminHandler {
@@ -52,7 +49,7 @@ public Optional<AssemblyEntity> getAssemblyByAccession(String accession) {
         return assemblyService.getAssemblyByAccession(accession);
     }
 
-    public void fetchAndInsertAssemblyByAccession(String accession) throws IOException {
+    public void fetchAndInsertAssemblyByAccession(String accession) {
         assemblyService.fetchAndInsertAssembly(accession);
     }
 
@@ -64,8 +61,20 @@ public void retrieveAndInsertMd5ChecksumForAssembly(String accession) {
         assemblyService.retrieveAndInsertMd5ChecksumForAssembly(accession);
     }
 
-    public Map<String, Set<String>> getMD5ChecksumUpdateTaskStatus() {
-        return assemblyService.getMD5ChecksumUpdateTaskStatus();
+    public void retrieveAndInsertMd5ChecksumForAssembly(List<String> accessions) {
+        assemblyService.retrieveAndInsertMd5ChecksumForAssembly(accessions);
+    }
+
+    public void retrieveAndInsertENASequenceNameForAssembly(String accession) {
+        assemblyService.retrieveAndInsertENASequenceNameForAssembly(accession);
+    }
+
+    public void retrieveAndInsertENASequenceNameForAssembly(List<String> accessions) {
+        assemblyService.retrieveAndInsertENASequenceNameForAssembly(accessions);
+    }
+
+    public List<String> getScheduledJobStatus() {
+        return assemblyService.getScheduledJobStatus();
     }
 
     public void deleteAssemblyByAccession(String accession) {

src/main/java/uk/ac/ebi/eva/contigalias/controller/contigalias/ContigAliasHandler.java

@@ -19,7 +19,6 @@
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.data.domain.Page;
 import org.springframework.data.domain.PageImpl;
-import org.springframework.data.domain.PageRequest;
 import org.springframework.data.domain.Pageable;
 import org.springframework.data.web.PagedResourcesAssembler;
 import org.springframework.hateoas.EntityModel;
@@ -83,6 +82,7 @@ public PagedModel<EntityModel<AssemblyEntity>> getAssemblyByRefseq(String refseq
 
     public PagedModel<EntityModel<AssemblyEntity>> getAssembliesByTaxid(long taxid, Pageable request) {
         Page<AssemblyEntity> page = assemblyService.getAssembliesByTaxid(taxid, request);
+        page.forEach(it->it.setChromosomes(null));
         return generatePagedModelFromPage(page, assemblyAssembler);
     }