Improve logging

bbimber · bbimber · commit d008049aa768 · 2024-10-23T21:53:32.000-07:00
diff --git a/mGAP/src/org/labkey/mgap/pipeline/RenameSamplesForMgapStep.java b/mGAP/src/org/labkey/mgap/pipeline/RenameSamplesForMgapStep.java
@@ -268,7 +268,7 @@ private Map<String, String> getSamplesToAlias(File input) throws PipelineJobExce
             sampleNames.removeAll(sampleNameMap.keySet());
             if (!sampleNames.isEmpty())
             {
-                throw new PipelineJobException("mGAP Aliases were not found for all IDs.  Missing: " + StringUtils.join(sampleNames, ", "));
+                throw new PipelineJobException("mGAP Aliases were not found for all IDs.  Missing: [" + StringUtils.join(sampleNames, ", ") + "]");
             }
 
             //Now ensure we dont have duplicate mappings:

Original file line number	Diff line number	Diff line change
`@@ -268,7 +268,7 @@ private Map<String, String> getSamplesToAlias(File input) throws PipelineJobExce`
`268`	`268`	`sampleNames.removeAll(sampleNameMap.keySet());`
`269`	`269`	`if (!sampleNames.isEmpty())`
`270`	`270`	`{`
`271`		`- throw new PipelineJobException("mGAP Aliases were not found for all IDs. Missing: " + StringUtils.join(sampleNames, ", "));`
	`271`	`+ throw new PipelineJobException("mGAP Aliases were not found for all IDs. Missing: [" + StringUtils.join(sampleNames, ", ") + "]");`
`272`	`272`	`}`
`273`	`273`
`274`	`274`	`//Now ensure we dont have duplicate mappings:`