Lets keep the error_state as is since it does not really do anything. Will remove this completely from the code in another PR.

Author: Xin Dong

src/QLContext.actor.cpp

@@ -353,6 +353,7 @@ struct CompoundIndexPlugin : IndexPlugin, ReferenceCounted<CompoundIndexPlugin>,
 			}
 
 			if (num_new_values > DOCLAYER_KNOBS->MULTI_MULTIKEY_INDEX_MAX) {
+				self->error_state = true;
 				throw multikey_index_cartesian_explosion();
 			}
 
@@ -519,6 +520,7 @@ struct SimpleIndexPlugin : IndexPlugin, ReferenceCounted<SimpleIndexPlugin>, Fas
 						    existing_index_entries.front().arena());
 						if (existingDocId.compare(documentPath[documentPath.size() - 1])) {
 							// existing index points to a different doc id that has the same value, abort.
+							self->error_state = true;
 							throw duplicated_key_field();
 						}
 					}