Elasticsearch: Suporte a arquivos de configuração do Cleanup Elasticsearch

Criado em 12 jul. 2016  ·  3Comentários  ·  Fonte: elastic/elasticsearch

Hoje, o Elasticsearch pode ser configurado por meio de arquivos YAML, JSON ou de propriedades. O suporte para arquivos de propriedades não é documentado e deve ser removido imediatamente. O suporte para arquivos JSON deve ser descontinuado e removido no 6.0.0.

  • [x] remove o suporte para .properties # 19398
  • [] descontinuar o suporte para .json em 2.4.0
  • [] remover o suporte para .json em 5.0.0
  • [] descontinuar o suporte .yml em 2.4.0
  • [] falha difícil se .yml existe no 5.0.0
  • [] remover verificação de existência de .yml em 6.0.0
:CorInfrSettings Meta good first issue help wanted

Comentários muito úteis

Também temos o caso estranho de apoiar tanto elasticsearch.yml quanto elasticsearch.yaml . O primeiro é o nome original e o último foi adicionado porque .yaml é a extensão oficial (de acordo com yaml.org). Embora eu saiba que .yml já existe há muito mais tempo, acho que devemos fazer com que nosso único nome de arquivo de configuração use .yaml . Além de ser a extensão oficial, eu também fui um pouco antes ao escrever testes de descanso e acidentalmente usar .yml , porque eu tinha acabado de editar um arquivo elasticsearch.yml .

Em todos os 3 casos (.yml, json e propriedades), acho que podemos fazer isso de forma limpa para o usuário, procurando os nomes de arquivo mais antigos na inicialização e gerando um erro. Na versão 2.0, já passávamos a falhar se houvesse mais de um arquivo disponível.

Todos 3 comentários

Também temos o caso estranho de apoiar tanto elasticsearch.yml quanto elasticsearch.yaml . O primeiro é o nome original e o último foi adicionado porque .yaml é a extensão oficial (de acordo com yaml.org). Embora eu saiba que .yml já existe há muito mais tempo, acho que devemos fazer com que nosso único nome de arquivo de configuração use .yaml . Além de ser a extensão oficial, eu também fui um pouco antes ao escrever testes de descanso e acidentalmente usar .yml , porque eu tinha acabado de editar um arquivo elasticsearch.yml .

Em todos os 3 casos (.yml, json e propriedades), acho que podemos fazer isso de forma limpa para o usuário, procurando os nomes de arquivo mais antigos na inicialização e gerando um erro. Na versão 2.0, já passávamos a falhar se houvesse mais de um arquivo disponível.

Embora eu saiba que .yml existe há muito mais tempo, acho que devemos fazer com que nosso único nome de arquivo de configuração use .yaml .

+1

Relates # 9706

Esta página foi útil?
0 / 5 - 0 avaliações