Elasticsearch: Prise en charge des fichiers de configuration de nettoyage Elasticsearch

Créé le 12 juil. 2016  ·  3Commentaires  ·  Source: elastic/elasticsearch

Aujourd'hui, Elasticsearch peut être configuré via YAML, JSON ou des fichiers de propriétés. La prise en charge des fichiers de propriétés n'est pas documentée et doit être immédiatement supprimée. La prise en charge des fichiers JSON doit être obsolète et supprimée dans la version 6.0.0.

  • [x] supprime la prise en charge de .properties #19398
  • [ ] désapprouve la prise en charge de .json dans la version 2.4.0
  • [ ] supprime la prise en charge de .json dans 5.0.0
  • [ ] déprécier le support .yml dans 2.4.0
  • [ ] échouer si .yml existe dans 5.0.0
  • [ ] supprime la vérification de l'existence de .yml dans 6.0.0
:CorInfrSettings Meta good first issue help wanted
Source
picture jasontedor
👍2

Commentaire le plus utile

Nous avons également le cas étrange de supporter à la fois elasticsearch.yml et elasticsearch.yaml . Le premier est le nom d'origine, et le dernier a été ajouté car .yaml est l'extension officielle (selon yaml.org). Bien que je comprenne que .yml existe depuis bien plus longtemps, je pense que nous devrions faire en sorte que notre nom de fichier de configuration unique utilise .yaml . En plus d'être l'extension officielle, j'ai également été un peu avant lors de l'écriture de tests de repos et en utilisant accidentellement .yml , car je venais d'éditer un fichier elasticsearch.yml .

Dans les 3 cas (.yml, json et propriétés), je pense que nous pouvons le faire proprement pour l'utilisateur en recherchant les anciens noms de fichiers au démarrage et en lançant une erreur. Dans la version 2.0, nous passions déjà à l'échec s'il y avait plus d'un fichier disponible.

picture rjernst le 12 juil. 2016
👍2

Tous les 3 commentaires

Nous avons également le cas étrange de supporter à la fois elasticsearch.yml et elasticsearch.yaml . Le premier est le nom d'origine, et le dernier a été ajouté car .yaml est l'extension officielle (selon yaml.org). Bien que je comprenne que .yml existe depuis bien plus longtemps, je pense que nous devrions faire en sorte que notre nom de fichier de configuration unique utilise .yaml . En plus d'être l'extension officielle, j'ai également été un peu avant lors de l'écriture de tests de repos et en utilisant accidentellement .yml , car je venais d'éditer un fichier elasticsearch.yml .

Dans les 3 cas (.yml, json et propriétés), je pense que nous pouvons le faire proprement pour l'utilisateur en recherchant les anciens noms de fichiers au démarrage et en lançant une erreur. Dans la version 2.0, nous passions déjà à l'échec s'il y avait plus d'un fichier disponible.

rjernst picture rjernst le 12 juil. 2016
👍2

Bien que je comprenne que .yml existe depuis bien plus longtemps, je pense que nous devrions faire en sorte que notre nom de fichier de configuration unique utilise .yaml .

+1

jasontedor picture jasontedor le 12 juil. 2016

Liens #9706

jasontedor picture jasontedor le 14 juil. 2016
Cette page vous a été utile?
0 / 5 - 0 notes

Questions connexes

clintongormley picture clintongormley  ·  3Commentaires
dadoonet picture dadoonet  ·  3Commentaires
DhairyashilBhosale picture DhairyashilBhosale  ·  3Commentaires
rbayliss picture rbayliss  ·  3Commentaires
ttaranov picture ttaranov  ·  3Commentaires