Skip to content

Commit

Permalink
[Test] Fix index.mapping.total_fields.limit random update (#119728)
Browse files Browse the repository at this point in the history
It's stupid to pick a low value for `index.mapping.total_fields.limit`
when at the same time you also add random runtime fields => pick a value
large enough to avoid failing future random mapping updates.

Closes #119646
Closes #119632
Closes #119631
  • Loading branch information
tlrx authored Jan 8, 2025
1 parent ee7848e commit 38260aa
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -207,7 +207,7 @@ protected static void updateRandomIndexSettings(String indexName) throws IOExcep
switch (i) {
case 0 -> settings.putList(IndexSettings.DEFAULT_FIELD_SETTING.getKey(), "field_" + randomInt(2));
case 1 -> settings.put(IndexSettings.MAX_INNER_RESULT_WINDOW_SETTING.getKey(), randomIntBetween(1, 100));
case 2 -> settings.put(MapperService.INDEX_MAPPING_TOTAL_FIELDS_LIMIT_SETTING.getKey(), randomLongBetween(0L, 1000L));
case 2 -> settings.put(MapperService.INDEX_MAPPING_TOTAL_FIELDS_LIMIT_SETTING.getKey(), randomLongBetween(100L, 1000L));
case 3 -> settings.put(IndexSettings.MAX_SLICES_PER_SCROLL.getKey(), randomIntBetween(1, 1024));
default -> throw new IllegalStateException();
}
Expand Down

0 comments on commit 38260aa

Please sign in to comment.