Alterações para reprocessamento do recurso SMTR202404004977 #124
Conversation
WalkthroughEsta alteração introduz um novo modelo SQL para ingestão de dados de vistoria, atualiza a agregação de licenciamento para incluir essa nova fonte, adiciona o novo recurso ao catálogo de fontes e documenta as mudanças no changelog. Outras modificações consistem apenas em ajustes de formatação e padronização em arquivos SQL existentes, sem alteração de lógica. Changes
Sequence Diagram(s)sequenceDiagram
participant FonteVistoria as Fonte de Dados de Vistoria
participant NovoModelo as Novo Modelo de Vistoria (staging)
participant Licenciamento as aux_sppo_licenciamento_vistoria_atualizada
FonteVistoria->>NovoModelo: Seleciona e tipa colunas (data, id_veiculo, placa, etc.)
NovoModelo->>Licenciamento: Fornece registros via UNION ALL
Licenciamento->>Licenciamento: Agrega e seleciona ano_ultima_vistoria máximo por veículo
Possibly related PRs
Poema
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
⏰ Context from checks skipped due to timeout of 90000ms (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (9)
queries/dbt_project.yml(2 hunks)queries/models/dashboard_subsidio_sppo/viagens_remuneradas.sql(1 hunks)queries/models/projeto_subsidio_sppo/deprecated/subsidio_shapes_geom.sql(1 hunks)queries/models/projeto_subsidio_sppo/subsidio_data_versao_efetiva.sql(1 hunks)queries/models/projeto_subsidio_sppo/viagem_planejada.sql(1 hunks)queries/models/sources.yml(1 hunks)queries/models/veiculo/sppo_veiculo_dia.sql(1 hunks)queries/models/veiculo/staging/aux_sppo_licenciamento_vistoria_atualizada.sql(1 hunks)queries/models/veiculo/staging/sppo_vistoria_tr_subtt_cmo_recurso_SMTR202404004977_staging.sql(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: Build and publish docs
🔇 Additional comments (10)
queries/models/dashboard_subsidio_sppo/viagens_remuneradas.sql (1)
1-457: Ajuste de formatação sem impacto funcional
Todas as modificações são mudanças de estilo (prefixos, indentação) e não alteram a lógica SQL existente.queries/models/projeto_subsidio_sppo/subsidio_data_versao_efetiva.sql (1)
1-706: Reformatação de Jinja apenas
As alterações se limitam à padronização da sintaxe de templating (curly braces) sem qualquer mudança de comportamento ou desempenho do modelo.queries/dbt_project.yml (3)
248-253: Validação de nomenclatura de schema para projeto_subsidio_sppo
O schema foi alterado paraSMTR202404004977_reprocessamento_projeto_subsidio_sppo. Confirme se esse padrão (prefixoSMTR202404004977_reprocessamento_) está correto e aplicado de forma consistente nos demais modelos do mesmo domínio.
269-272: Atualização de schema em dashboard_subsidio_sppo
Ajustado paraSMTR202404004977_reprocessamento_dashboard_subsidio_sppo. Verifique se não há referências diretas a este modelo em testes ou macros que precisem do schema antigo.
275-280: Configuração de schema em veiculo e staging
Os schemasSMTR202404004977_reprocessamento_veiculoeSMTR202404004977_reprocessamento_veiculo_stagingforam adicionados. Assegure-se de que as fontes (sources) e unions emaux_sppo_licenciamento_vistoria_atualizada.sqlforam atualizadas para refletir esses novos schemas.queries/models/projeto_subsidio_sppo/deprecated/subsidio_shapes_geom.sql (1)
29-32: Remoção da condição incremental no WHERE
A filtragem pordata_versaoagora é aplicada em todas as execuções (full+incremental). Confirme se este comportamento incondicional é intencional para o reprocessamento ou se impacta a carga histórica.queries/models/projeto_subsidio_sppo/viagem_planejada.sql (1)
1-7: Padronização do bloco de configuração Jinja
As alterações no blococonfig(...)são apenas de formatação (estilo de chaves e indentação), sem impacto na lógica SQL.queries/models/sources.yml (1)
124-124: Comentário de source desativado intencionalmente
O tablesppo_vistoria_tr_subtt_cmo_recurso_SMTR202404004977foi comentado para migrar o fluxo ao modelo de staging dedicado. Confirme que não restam referências ao source original em outros artefatos.queries/models/veiculo/sppo_veiculo_dia.sql (1)
330-330: Nenhum comentário necessário
A única modificação é remoção de linha em branco ao final do arquivo, sem impacto na lógica.queries/models/veiculo/staging/aux_sppo_licenciamento_vistoria_atualizada.sql (1)
40-47: Inclusão de novo staging model na união
OUNION ALLque referenciasppo_vistoria_tr_subtt_cmo_recurso_SMTR202404004977_stagingalinha-se à desativação do source original e ao novo fluxo de staging.
queries/models/veiculo/staging/sppo_vistoria_tr_subtt_cmo_recurso_SMTR202404004977_staging.sql
Outdated
Show resolved
Hide resolved
eng-rodrigocunha
changed the title
|
@vtr363 esse pull request tem conflitos 😩 |
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
queries/models/veiculo/CHANGELOG.md (2)
5-8: Padronize o estilo da lista e evite MD034 (bare URL).
- Mantendo a consistência com entradas anteriores do changelog, o item em “Adicionado” deve iniciar com
-.- O link nu viola a regra MD034 do markdownlint; envolva-o em
< >ou use a sintaxe[texto](url).-Criado o modelo `sppo_vistoria_tr_subtt_cmo_recurso_SMTR202404004977_staging.sql` (https://github.com/prefeitura-rio/pipelines_rj_smtr/pull/124) +- Criado o modelo `sppo_vistoria_tr_subtt_cmo_recurso_SMTR202404004977_staging.sql` (<https://github.com/prefeitura-rio/pipelines_rj_smtr/pull/124>)
11-11: Uniformize o link e corrija aviso de lint.O hyperlink nu aqui também dispara MD034. Aproveite para manter o mesmo formato adotado na sugestão anterior.
-- Alterado o modelo `aux_sppo_licenciamento_vistoria_atualizada` para incluir o modelo `sppo_vistoria_tr_subtt_cmo_recurso_SMTR202404004977_staging.sql` (https://github.com/prefeitura-rio/pipelines_rj_smtr/pull/124) +- Alterado o modelo `aux_sppo_licenciamento_vistoria_atualizada` para incluir o modelo `sppo_vistoria_tr_subtt_cmo_recurso_SMTR202404004977_staging.sql` (<https://github.com/prefeitura-rio/pipelines_rj_smtr/pull/124>)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
pipelines/migration/projeto_subsidio_sppo/flows.py(1 hunks)pipelines/migration/veiculo/flows.py(1 hunks)queries/models/veiculo/CHANGELOG.md(1 hunks)
✅ Files skipped from review due to trivial changes (2)
- pipelines/migration/veiculo/flows.py
- pipelines/migration/projeto_subsidio_sppo/flows.py
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
queries/models/veiculo/CHANGELOG.md
7-7: Bare URL used
null
(MD034, no-bare-urls)
11-11: Bare URL used
null
(MD034, no-bare-urls)
⏰ Context from checks skipped due to timeout of 90000ms (2)
- GitHub Check: WIP
- GitHub Check: WIP
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
queries/models/veiculo/CHANGELOG.md (2)
7-7: Padronizar item da lista e evitar bare URLEste item não segue o padrão dos demais (falta o marcador
-) e expõe a URL sem formatação, violando a regra MD034.Sugestão de correção:
-Criado o modelo `sppo_vistoria_tr_subtt_cmo_recurso_SMTR202404004977_staging.sql` (https://github.com/prefeitura-rio/pipelines_rj_smtr/pull/124) +- Criado o modelo `sppo_vistoria_tr_subtt_cmo_recurso_SMTR202404004977_staging.sql` ([#124](https://github.com/prefeitura-rio/pipelines_rj_smtr/pull/124))
11-11: Formatação de link MarkdownA URL também está exposta como bare URL. Para manter a consistência e evitar alertas do markdownlint:
-- Alterado o modelo `aux_sppo_licenciamento_vistoria_atualizada` para incluir o modelo `sppo_vistoria_tr_subtt_cmo_recurso_SMTR202404004977_staging.sql` (https://github.com/prefeitura-rio/pipelines_rj_smtr/pull/124) +- Alterado o modelo `aux_sppo_licenciamento_vistoria_atualizada` para incluir o modelo `sppo_vistoria_tr_subtt_cmo_recurso_SMTR202404004977_staging.sql` ([#124](https://github.com/prefeitura-rio/pipelines_rj_smtr/pull/124))
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
queries/models/dashboard_subsidio_sppo/viagens_remuneradas.sql(1 hunks)queries/models/projeto_subsidio_sppo/viagem_planejada.sql(1 hunks)queries/models/sources.yml(1 hunks)queries/models/veiculo/CHANGELOG.md(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (3)
- queries/models/sources.yml
- queries/models/projeto_subsidio_sppo/viagem_planejada.sql
- queries/models/dashboard_subsidio_sppo/viagens_remuneradas.sql
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: akaBotelho
PR: prefeitura-rio/pipelines_rj_smtr#624
File: queries/models/dashboard_subsidio_sppo_v2/schema.yml:231-245
Timestamp: 2025-06-24T16:12:05.914Z
Learning: Nos modelos de subsídio do projeto pipelines_rj_smtr, as colunas de quilometragem (km_apurada_*) utilizam COALESCE para definir valores nulos como 0, tornando testes de not_null e accepted_range com min_value: 0 redundantes no schema.yml.
Learnt from: akaBotelho
PR: prefeitura-rio/pipelines_rj_smtr#595
File: queries/models/monitoramento/staging/aux_gps_trajeto_correto.sql:45-47
Timestamp: 2025-05-27T20:44:58.877Z
Learning: A variável `buffer_segmento_metros` está definida no arquivo `queries/dbt_project.yml` na seção "### Viagens 2.0 ###" com valor 20, e é usada corretamente nos modelos dbt via `{{ var("buffer_segmento_metros") }}`.
queries/models/veiculo/CHANGELOG.md (2)
Learnt from: akaBotelho
PR: prefeitura-rio/pipelines_rj_smtr#624
File: queries/models/dashboard_subsidio_sppo_v2/schema.yml:231-245
Timestamp: 2025-06-24T16:12:05.914Z
Learning: Nos modelos de subsídio do projeto pipelines_rj_smtr, as colunas de quilometragem (km_apurada_*) utilizam COALESCE para definir valores nulos como 0, tornando testes de not_null e accepted_range com min_value: 0 redundantes no schema.yml.
Learnt from: akaBotelho
PR: prefeitura-rio/pipelines_rj_smtr#595
File: queries/models/monitoramento/staging/aux_gps_trajeto_correto.sql:45-47
Timestamp: 2025-05-27T20:44:58.877Z
Learning: A variável `buffer_segmento_metros` está definida no arquivo `queries/dbt_project.yml` na seção "### Viagens 2.0 ###" com valor 20, e é usada corretamente nos modelos dbt via `{{ var("buffer_segmento_metros") }}`.
🪛 markdownlint-cli2 (0.17.2)
queries/models/veiculo/CHANGELOG.md
7-7: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: WIP
- GitHub Check: Build and publish docs
- GitHub Check: WIP
|
@coderabbitai full review |
✅ Actions performedFull review triggered. |
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Análise da árvore de códigoNenhum problema encontrado! |
eng-rodrigocunha
changed the title
Summary by CodeRabbit
Novos Recursos
Documentação
Chores