DocValuesConsumerUtil tweak

Be more strict about not finding a metadata entry based on field info number.

If this were to happen, then this can cause issues in doc value consumer. However as far as I understand it, the field info number should stay consistent.

Author: martijnvg

server/src/main/java/org/elasticsearch/index/codec/tsdb/es819/DocValuesConsumerUtil.java

@@ -60,20 +60,29 @@ static MergeStats compatibleWithOptimizedMerge(boolean optimizedMergeEnabled, Me
                             if (entry != null) {
                                 sumNumValues += entry.numValues;
                                 sumNumDocsWithField += entry.numDocsWithField;
+                            } else {
+                                assert false : "unexpectedly got no entry for field [" + fieldInfo.number + "\\" + fieldInfo.name + "]";
+                                return UNSUPPORTED;
                             }
                         }
                         case SORTED_NUMERIC -> {
                             var entry = tsdbDocValuesProducer.sortedNumerics.get(fieldInfo.number);
                             if (entry != null) {
                                 sumNumValues += entry.numValues;
                                 sumNumDocsWithField += entry.numDocsWithField;
+                            } else {
+                                assert false : "unexpectedly got no entry for field [" + fieldInfo.number + "\\" + fieldInfo.name + "]";
+                                return UNSUPPORTED;
                             }
                         }
                         case SORTED -> {
                             var entry = tsdbDocValuesProducer.sorted.get(fieldInfo.number);
                             if (entry != null) {
                                 sumNumValues += entry.ordsEntry.numValues;
                                 sumNumDocsWithField += entry.ordsEntry.numDocsWithField;
+                            } else {
+                                assert false : "unexpectedly got no entry for field [" + fieldInfo.number + "\\" + fieldInfo.name + "]";
+                                return UNSUPPORTED;
                             }
                         }
                         case SORTED_SET -> {
@@ -86,6 +95,9 @@ static MergeStats compatibleWithOptimizedMerge(boolean optimizedMergeEnabled, Me
                                     sumNumValues += entry.ordsEntry.numValues;
                                     sumNumDocsWithField += entry.ordsEntry.numDocsWithField;
                                 }
+                            } else {
+                                assert false : "unexpectedly got no entry for field [" + fieldInfo.number + "\\" + fieldInfo.name + "]";
+                                return UNSUPPORTED;
                             }
                         }
                         case BINARY -> {
@@ -94,6 +106,9 @@ static MergeStats compatibleWithOptimizedMerge(boolean optimizedMergeEnabled, Me
                                 sumNumDocsWithField += entry.numDocsWithField;
                                 minLength = Math.min(minLength, entry.minLength);
                                 maxLength = Math.max(maxLength, entry.maxLength);
+                            } else {
+                                assert false : "unexpectedly got no entry for field [" + fieldInfo.number + "\\" + fieldInfo.name + "]";
+                                return UNSUPPORTED;
                             }
                         }
                         default -> throw new IllegalStateException("unexpected doc values producer type: " + fieldInfo.getDocValuesType());