Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Substituição da biblioteca python elasticsearch pelo opensearch #65

Merged
merged 1 commit into from
Dec 6, 2023

Conversation

tigreped
Copy link
Collaborator

@tigreped tigreped commented Nov 8, 2023

…te opensearch, com atualização da dependência requests para compatibilidade com a biblioteca mais atual do opensearch. Adição ao gitignore para evitar subir arquivos em contexto de desenvolvimento local.

Copy link
Member

@ogecece ogecece left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tigreped ainda precisamos ajustar algumas coisas e tenho a impressão que coisas que não deveriam ter subido no PR foram subidas. Pode avaliar aqui?

Ah, e pode fazer um squash desse PR pra um commit apenas e manter a mensagem do commit num padrão básico de:

  • Título
    • Até 72 caracteres (limite rígido) mas preferencialmente até 50 (limite flexível)
    • Responde "O que esse commit faz?"
    • Ex: "Substitui cliente elasticsearch por opensearch"
  • Descrição
    • Até 72 caracteres (limite flexível)
    • Responde "Por que estamos fazendo essas modificações?"
    • Se precisar complementar o "o que esse commit faz" com alguns pontos, pode complementar no começo da descrição

.gitignore Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
gazettes/gazette_access.py Outdated Show resolved Hide resolved
gazettes/gazette_access.py Outdated Show resolved Hide resolved
index/opensearch.py Outdated Show resolved Hide resolved
tests/test_opensearch.py Outdated Show resolved Hide resolved
tests/test_opensearch.py Outdated Show resolved Hide resolved
Copy link
Member

@ogecece ogecece left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Esqueci disso aqui

Makefile Outdated Show resolved Hide resolved
@tigreped tigreped changed the title Substituição da biblioteca do cliente python elasticsearch pelo clien… Substituição da biblioteca python elasticsearch pelo opensearch Nov 29, 2023
Alterações feitas para refletir todas as alterações de ES para OS. Falta apenas o Makefile.

Makefile adaptado para Opensearch

Correções diversas.
@ogecece ogecece merged commit 7bb0b2e into main Dec 6, 2023
1 check passed
@ogecece ogecece deleted the issue_64 branch December 6, 2023 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants