Elasticsearch: Unterstützung für die Bereinigung von Elasticsearch-Konfigurationsdateien

Erstellt am 12. Juli 2016  ·  3Kommentare  ·  Quelle: elastic/elasticsearch

Heute kann Elasticsearch über YAML, JSON oder Eigenschaftendateien konfiguriert werden. Die Unterstützung für Eigenschaftendateien ist nicht dokumentiert und sollte sofort entfernt werden. Die Unterstützung für JSON-Dateien sollte in 6.0.0 veraltet sein und entfernt werden.

  • [x] Entfernen Sie die Unterstützung für .properties #19398
  • [ ] veraltet die Unterstützung für .json in 2.4.0
  • [ ] Entfernen Sie die Unterstützung für .json in 5.0.0
  • [ ] unterstützt .yml in 2.4.0 nicht mehr unterstützt
  • [ ] schlägt hart fehl, wenn .yml in 5.0.0 vorhanden ist
  • [ ] Prüfung auf Existenz von .yml in 6.0.0 entfernen
:CorInfrSettings Meta good first issue help wanted

Hilfreichster Kommentar

Wir haben auch den seltsamen Fall, dass sowohl elasticsearch.yml als auch elasticsearch.yaml . Ersteres ist der ursprüngliche Name und letzteres wurde hinzugefügt, weil .yaml die offizielle Erweiterung ist (laut yaml.org). Obwohl ich verstehe, dass es .yml schon viel länger gibt, denke ich, dass wir für unseren einzigen Konfigurationsdateinamen .yaml . Abgesehen davon, dass es die offizielle Erweiterung ist, war ich auch schon beim Schreiben von Ruhetests und der versehentlichen Verwendung von .yml , weil ich gerade eine elasticsearch.yml Datei bearbeitet hatte.

In allen 3 Fällen (.yml, json und properties) können wir dies meiner Meinung nach für den Benutzer sauber tun, indem wir beim Start nach den älteren Dateinamen suchen und einen Fehler ausgeben. In 2.0 haben wir bereits auf Fehler umgeschaltet, wenn mehr als eine Datei verfügbar war.

Alle 3 Kommentare

Wir haben auch den seltsamen Fall, dass sowohl elasticsearch.yml als auch elasticsearch.yaml . Ersteres ist der ursprüngliche Name und letzteres wurde hinzugefügt, weil .yaml die offizielle Erweiterung ist (laut yaml.org). Obwohl ich verstehe, dass es .yml schon viel länger gibt, denke ich, dass wir für unseren einzigen Konfigurationsdateinamen .yaml . Abgesehen davon, dass es die offizielle Erweiterung ist, war ich auch schon beim Schreiben von Ruhetests und der versehentlichen Verwendung von .yml , weil ich gerade eine elasticsearch.yml Datei bearbeitet hatte.

In allen 3 Fällen (.yml, json und properties) können wir dies meiner Meinung nach für den Benutzer sauber tun, indem wir beim Start nach den älteren Dateinamen suchen und einen Fehler ausgeben. In 2.0 haben wir bereits auf Fehler umgeschaltet, wenn mehr als eine Datei verfügbar war.

Obwohl ich verstehe, dass es .yml schon viel länger gibt, denke ich, dass wir für unseren einzigen Konfigurationsdateinamen .yaml .

+1

Bezieht sich auf #9706

War diese Seite hilfreich?
0 / 5 - 0 Bewertungen