elastic/elasticsearch

Detect when copying to a non-existent field when dynamic mappings are disabled

Open

#112.812 geöffnet am 12. Sept. 2024

Auf GitHub ansehen
 (20 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Java (76.700 Stars) (25.882 Forks)batch import
:Search Foundations/Mapping:Search Relevance/Search>enhancementTeam:Search FoundationsTeam:Search Relevancegood first issue

Beschreibung

Description

We should be able to detect when a user is attempting to use copy_to to copy to a non-existent field when dynamic mappings are disabled. For example:

PUT test-index
{
  "mappings": {
    "dynamic": false,
    "properties": {
      "test_field": {
        "type": "text",
        "copy_to": "missing_field"
      }
    }
  }
}

Currently, this request succeeds even though nothing will ever be copied to missing_field. We should throw a 400 error in this case to indicate to the user that their mappings are invalid.

Contributor Guide

Tech Stack
javarest api
Domain
backendapi
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
half day
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
active
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
JavaElasticsearch mapping conceptsUnderstanding of dynamic mapping
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
45
Research-Richtung
Investigate the mapping parsing code where copy to is handled, likely in classes like MappingMetaData or FieldMapper. Examine the logic that validates field existence when dynamic is false. Consider how to integrate a check that throws an exception. Review the existing test suite for mapping validation to understand patterns. Check the issue comments for any suggested approaches.