-
Notifications
You must be signed in to change notification settings - Fork 1
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
[WIP] Refatora recapturas de GPS do SPPO #168
Conversation
===> 1 warning(s) found:
|
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
===> 1 warning(s) found:
|
1 similar comment
===> 1 warning(s) found:
|
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
===> 1 warning(s) found:
|
===> 1 warning(s) found:
|
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
@@ -83,6 +85,9 @@ def pre_treatment_br_rj_riodejaneiro_onibus_gps( | |||
timezone = constants.TIMEZONE.value | |||
|
|||
log(f"Data received to treat: \n{status['data'][:5]}") | |||
if status["data"] == []: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remover, ja esta na linha 80
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tudo parece OK. Basta fechar os testes para subir.
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
Análise da árvore de códigoOs seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request: Os seguintes usuários devem ser avisados sobre a alteração: |
@Hellcassius esse pull request tem conflitos 😩 |
@Hellcassius esse pull request tem conflitos 😩 |
(1) Refatoração do flow
captura_sppo_v2
: Ajustes para uso dacaptura_sppo_v2
na recaptura *Changelog:
timestamp
- quando há recaptura, utiliza parametro opcionaldatetime_filter
, cc usa timestamp atual; fixa formatacao emisoformat
upload_logs_to_bq
: mantém log do erro anterior que foi recapturadorun_config
: limita uso de cpu e memoria(2) Refatoração do flow
recaptura
: Ajustes para uso dacaptura_sppo_v2
na recapturaChangelog:
create_flow_run
: usa o flow decaptura_sppo_v2
para criar múltiplas runs do mesmo, substituindo o uso do método .map em cada taskquery_logs
: aumenta limite de 40 para 60 recapturas por run; retorna dicionário com timestamp e erro + flag de recaptura, novos parametros que são passados nocreate_flow_run
para acaptura_sppo_v2
run_config
: limita uso de cpu e memoria(3) Refatoração do flow
materialize
: Corrije atraso na materialização quando há falhasChangelog:
get_materialization_date_range
: parametriza o delay da materialização de acordo com o flow + variastart_ts
para preencher buracos de materialização (caso haja algum), fixandoend_ts
com base nodelay_hour