Skip to content

Commit 1f2e0c0

Browse files
committed
storage: Ensure updated documents have at least one variantStats. #TASK-6217
1 parent b37b15e commit 1f2e0c0

File tree

2 files changed

+14
-1
lines changed

2 files changed

+14
-1
lines changed

opencga-storage/opencga-storage-core/src/main/java/org/opencb/opencga/storage/core/variant/search/VariantSearchSyncInfo.java

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,5 +105,13 @@ public static Status from(String c) {
105105
}
106106
}
107107

108-
108+
@Override
109+
public String toString() {
110+
final StringBuilder sb = new StringBuilder("VariantSearchSyncInfo{");
111+
sb.append("status=").append(status);
112+
sb.append(", studies=").append(studies);
113+
sb.append(", statsHash=").append(statsHash);
114+
sb.append('}');
115+
return sb.toString();
116+
}
109117
}

opencga-storage/opencga-storage-core/src/main/java/org/opencb/opencga/storage/core/variant/search/VariantToSolrBeanConverterTask.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,6 +78,11 @@ public VariantSearchUpdateDocument convert(Variant variant) {
7878
updateDocument.removeField(fieldName);
7979
}
8080
}
81+
if (updateDocument.getFieldNames().size() <= 1) {
82+
throw new IllegalStateException("No stats fields found in variant " + variant + ". "
83+
+ "This should not happen, as the variant is expected to have stats. "
84+
+ "Resolved sync info: " + resolvedSyncInfo);
85+
}
8186

8287
return new VariantSearchUpdateDocument(variant, resolvedSyncInfo, updateDocument, false);
8388
}

0 commit comments

Comments
 (0)