Improve error reporting for VcfToLuceneIndexer (#300)

BimberLab · Jan 4, 2024 · 6f9c818 · 6f9c818
1 parent dbd365e
commit 6f9c818
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/src/main/java/com/github/discvrseq/walkers/VcfToLuceneIndexer.java b/src/main/java/com/github/discvrseq/walkers/VcfToLuceneIndexer.java
@@ -298,12 +298,12 @@ else if (line.getCountType() == VCFHeaderLineCount.INTEGER || line.getCountType(
                     // should never reach this
                     throw new GATKException("Attempted to parse a numeric value into something other than Double/Integer. This should never occur.");
                 } catch (Exception e) {
-                    possiblyReportBadValue(key, valueStr);
+                    possiblyReportBadValue(e, key, valueStr);
                     return null;
                 }
             }
             else {
-                possiblyReportBadValue(key, valueStr);
+                possiblyReportBadValue(null, key, valueStr);
                 return null;
             }
         }
@@ -319,16 +319,16 @@ else if (clazz == Integer.class) {
                 throw new GATKException("Attempted to parse a numeric value into something other than Double/Integer. This should never occur.");
             }
             catch (Exception e) {
-                possiblyReportBadValue(key, valueStr);
+                possiblyReportBadValue(e, key, valueStr);
                 return null;
             }
         }
     }
 
     private final Set<String> keysWithErrors = new HashSet<>();
 
-    private void possiblyReportBadValue(String key, Object fieldValue) {
-        String message = "Unable to parse numeric value for field: " + key + ", was: <" + fieldValue + ">";
+    private void possiblyReportBadValue(@Nullable Exception e, String key, Object fieldValue) {
+        String message = "Unable to parse field: " + key + ", was: <" + fieldValue + ">. " + (e == null ? "" : e.getMessage());
         if (stringency == ValidationStringency.STRICT) {
             throw new GATKException(message);
         }
@@ -345,7 +345,7 @@ synchronized private void addFieldToDocument(Document doc, VCFHeaderLineType var
             stats.inspectValue(key, fieldValue);
         }
         catch (GATKException e) {
-            possiblyReportBadValue(key, fieldValue);
+            possiblyReportBadValue(e, key, fieldValue);
         }
 
         Collection<?> values = fieldValue instanceof Collection ? (Collection<?>) fieldValue : Collections.singleton(fieldValue);
@@ -377,6 +377,7 @@ synchronized private void addFieldToDocument(Document doc, VCFHeaderLineType var
                         }
                     }
                     case String -> doc.add(new TextField(key, String.valueOf(value), Field.Store.YES));
+                    default -> possiblyReportBadValue(new Exception("VCF header type was not expected: " + variantHeaderLineType.name()), key, value);
                 }
             }
             catch (Exception e) {