Improve logging

bbimber · bbimber · commit e106d9ce5f65 · 2023-01-09T15:40:43.000-08:00
diff --git a/SequenceAnalysis/src/org/labkey/sequenceanalysis/pipeline/SequenceOutputHandlerRemoteTask.java b/SequenceAnalysis/src/org/labkey/sequenceanalysis/pipeline/SequenceOutputHandlerRemoteTask.java
@@ -112,6 +112,7 @@ public static void possiblyCacheGenomes(SequenceJob job, List<SequenceOutputFile
     }
 
     @NotNull
+    @Override
     public RecordedActionSet run() throws PipelineJobException
     {
         SequenceTaskHelper.logModuleVersions(getJob().getLogger());
@@ -122,6 +123,7 @@ public RecordedActionSet run() throws PipelineJobException
         possiblyCacheGenomes(getPipelineJob(), getPipelineJob().getFiles());
 
         getJob().setStatus(PipelineJob.TaskStatus.running, "Running: " + handler.getName());
+        getJob().getLogger().info("Output file IDs: " + getPipelineJob().getFiles().stream().map(SequenceOutputFile::getRowid).map(String::valueOf).collect(Collectors.joining(",")));
         handler.getProcessor().processFilesRemote(getPipelineJob().getFiles(), ctx);
 
         //Note: on job resume the TaskFileManager could be replaced with one from the resumer

Original file line number	Diff line number	Diff line change
`@@ -112,6 +112,7 @@ public static void possiblyCacheGenomes(SequenceJob job, List<SequenceOutputFile`
`112`	`112`	`}`
`113`	`113`
`114`	`114`	`@NotNull`
	`115`	`+ @Override`
`115`	`116`	`public RecordedActionSet run() throws PipelineJobException`
`116`	`117`	`{`
`117`	`118`	`SequenceTaskHelper.logModuleVersions(getJob().getLogger());`
`@@ -122,6 +123,7 @@ public RecordedActionSet run() throws PipelineJobException`
`122`	`123`	`possiblyCacheGenomes(getPipelineJob(), getPipelineJob().getFiles());`
`123`	`124`
`124`	`125`	`getJob().setStatus(PipelineJob.TaskStatus.running, "Running: " + handler.getName());`
	`126`	`+ getJob().getLogger().info("Output file IDs: " + getPipelineJob().getFiles().stream().map(SequenceOutputFile::getRowid).map(String::valueOf).collect(Collectors.joining(",")));`
`125`	`127`	`handler.getProcessor().processFilesRemote(getPipelineJob().getFiles(), ctx);`
`126`	`128`
`127`	`129`	`//Note: on job resume the TaskFileManager could be replaced with one from the resumer`