From 8e0cafdd6180dac19a02dd30d0d3c0e085b8dbe9 Mon Sep 17 00:00:00 2001 From: Alex Chan Date: Tue, 20 Oct 2020 14:28:44 +0100 Subject: [PATCH] The ElasticIndexer should never be asked to index an empty list --- .../scala/uk/ac/wellcome/pipeline_storage/ElasticIndexer.scala | 1 + 1 file changed, 1 insertion(+) diff --git a/common/pipeline_storage/src/main/scala/uk/ac/wellcome/pipeline_storage/ElasticIndexer.scala b/common/pipeline_storage/src/main/scala/uk/ac/wellcome/pipeline_storage/ElasticIndexer.scala index f6e5d46fee..60aec4d6e6 100644 --- a/common/pipeline_storage/src/main/scala/uk/ac/wellcome/pipeline_storage/ElasticIndexer.scala +++ b/common/pipeline_storage/src/main/scala/uk/ac/wellcome/pipeline_storage/ElasticIndexer.scala @@ -22,6 +22,7 @@ class ElasticIndexer[T: Indexable]( new ElasticsearchIndexCreator(client, index, config).create final def index(documents: Seq[T]): Future[Either[Seq[T], Seq[T]]] = { + assert(documents.nonEmpty, "Cannot index an empty list of documents") debug(s"Indexing ${documents.map(doc => indexable.id(doc)).mkString(", ")}")