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

Melhoria no Monitoramento Automático #266

Merged
merged 8 commits into from May 29, 2017

Conversation

Projects
None yet
2 participants
@rafaelpatro
Contributor

rafaelpatro commented May 27, 2017

O Monitoramento/Tracking Automático das encomendas é realizado através das tarefas do Cron. Atualmente o módulo consulta cada encomenda separadamente, levando um tempo considerável para a conclusão da tarefa.

Exemplo 1:

  • A loja possui 180 pedidos em trânsito.
  • Está um dia bonito, e os Correios respondem em 1s.
  • O Magento salva as informações em 1s.
    O Monitoramento Automático concluirá a tarefa em 6min.

Exemplo 2:

  • A loja possui 180 pedidos em trânsito.
  • Está chovendo, e os Correios respondem em 2s.
  • O Magento salva as informações em 1s.
    O Monitoramento Automático concluirá a tarefa em 9min.

É fácil perceber que há um gargalo, e o monitoramento não foi dimensionado para grandes quantidades, durante seu desenvolvimento. Logo, segue proposta de modificação que inclui:

  • Reunir todos os códigos de rastreamento para uma consulta única.
  • Reunir todas as informações consultadas para salvar em uma transação única.

Exemplo (após a mudança):

  • A loja possui 300 pedidos em trânsito.
  • Está passando um furacão, e os Correios respondem em 20s.
  • O Magento salva as informações em 10s.
    O Monitoramento Automático concluirá a tarefa em 30s.

@pedro-teixeira pedro-teixeira self-assigned this May 27, 2017

@pedro-teixeira pedro-teixeira added this to the v4.9.0 milestone May 27, 2017

@pedro-teixeira

O build está quebrado

@@ -0,0 +1,67 @@
<?php
include_once('Rastro/BuscaEventosLista.php');

This comment has been minimized.

@pedro-teixeira

pedro-teixeira May 27, 2017

Owner

Não deveria ser preciso usar include ou require, não? O Magento deveria fazer o autoload corretamente.

This comment has been minimized.

@rafaelpatro

rafaelpatro May 27, 2017

Contributor

Realmente! Nem percebi.
Usei um gerador automático de classes wsdl, mas provavelmente o include é desnecessário mesmo. Vou remover assim que der.

rafaelpatro added some commits May 27, 2017

rafaelpatro added some commits May 27, 2017

@rafaelpatro

This comment has been minimized.

Contributor

rafaelpatro commented May 28, 2017

@pedro-teixeira, tem como dar uma atualizada nas regras do build?

Cenário 1:

image
image

Cenário 2:

image
image

Cenário 3:

image
image

@pedro-teixeira

@rafaelpatro vou fazer uma série de modificações no code sniffers no meu repo, por favor faz o merge no seu código

@pedro-teixeira pedro-teixeira merged commit 4baccbe into pedro-teixeira:master May 29, 2017

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment