Programação · tema 24 de 40
Git e revisão de código para quem trabalha com dados
30 min · 2 vídeos · 10 cards · 1 drill
Por que cai
A ementa pede Git, mas a banca pergunta outra coisa: como o seu trabalho vira algo que outra pessoa consegue revisar, repetir e reverter. Quem responde citando comandos perde; quem descreve o fluxo, o que escreve no pedido de revisão e o que olha ao revisar SQL alheio mostra que trabalha em equipe.
Pré-teste · 1 de 2
responda antes de verO que uma boa mensagem de commit deve conter?
Vídeo · português · 31 min
Em uma frase
O fluxo existe para que a sua mudança seja revisável, repetível e reversível — e, em dados, o revisor humano cuida do que nenhuma automação pega: grão, regra de negócio e impacto em quem consome.
O fluxo, e o porquê de cada passo
| Passo | Para que serve |
|---|---|
| Branch por mudança | Isolar o trabalho e permitir reverter só aquilo depois |
| Commits pequenos com intenção | Explicar o porquê, que o diff não mostra |
| Pedido de revisão com contexto | Dar ao revisor o que ele precisa para avaliar impacto |
| Verificação automática | Sintaxe, construção isolada, testes de dado |
| Revisão humana | Grão, regra de negócio, impacto, cobertura de teste |
| Integração e promoção | Mesmo artefato avançando entre ambientes |
O que escrever num pedido de revisão de dados
Três perguntas, e elas são diferentes das de um projeto de aplicação:
- O que muda no número? Se a definição de um indicador mudou, isso é a informação mais importante do pedido.
- Quem consome o modelo afetado? Vem da linhagem, e define quem precisa ser avisado.
- Como você validou? Testes que rodaram, comparação do total com a origem, o que mais você conferiu.
Automação e pessoa: dividir por competência
| Quem pega | O quê |
|---|---|
| Ferramenta | Sintaxe, formatação, construção isolada, testes de dado, cobertura |
| Pessoa | Grão mudou? Regra bate com a definição do negócio? Quem quebra? Falta teste? |
Duas armadilhas específicas de dados
Credencial no repositório. O histórico é permanente: apagar num commit posterior não remove do histórico, e quem clonou antes já tem. Segredo vai para cofre, sempre.
Notebook versionado sem cuidado. Ele guarda saídas e a ordem em que as células foram executadas, o que polui o diff e esconde estado. Limpe as saídas antes de commitar, ou promova a lógica estável para um módulo e deixe o notebook como exploração.
Como responder isso em voz alta
Descreva o fluxo, não os comandos. Diga o que vai na descrição do pedido de revisão em contexto de dados. E separe explicitamente o que a automação verifica do que a pessoa revisa — é essa separação que mostra que você já revisou o trabalho de alguém, e não só leu sobre o assunto.
Se quiser outro ângulo1 vídeo
Como cai na sabatina
“Como uma mudança sua numa transformação chega ao repositório e ao ambiente dos outros?”
Erros comuns
- Commitar na branch principal direto. Perde revisão, perde o momento em que alguém poderia ter visto o erro, e dificulta reverter só aquela mudança.
- Escrever mensagem de commit que descreve o diff. 'Altera consulta' não diz nada; a mensagem existe para explicar a intenção e o porquê, que o diff não mostra.
- Abrir pedido de revisão gigante. Revisão de 800 linhas vira aprovação sem leitura; mudança pequena e frequente é revisada de verdade.
- Revisar SQL procurando erro de sintaxe. Isso o computador faz; o revisor humano existe para checar grão, regra de negócio e impacto em quem consome.
- Versionar credencial junto com o código. Repositório é histórico permanente; apagar em commit posterior não apaga do histórico.
- Tratar notebook como entregável versionado sem cuidado. Notebook guarda saída e ordem de execução, o que polui o diff e esconde estado; o que vai para o repositório precisa ser limpo ou convertido.
Flashcards
Card 1 de 10
0 certos · 0 a reverDrill
Automação ou pessoa?
Para cada item, diga quem deveria pegar isso.
1.Indentação e formatação do SQL.
2.A junção mudou o grão da tabela.
3.Coluna obrigatória ficou nula na saída.
4.A definição de cliente ativo mudou sem avisar a área de negócio.
5.O SQL tem erro de sintaxe.
Quiz · 1 de 2
Qual prática de revisão de código é mais defensável num time de dados?
Explique para um gerente
Explique em um minuto por que salvar 'consulta_final_v3_ok.sql' na sua máquina é pior que um histórico de versões, mesmo trabalhando sozinho.
Se travar numa palavra técnica, é sinal de que ainda não entendeu essa parte.
Prefiro falar em voz alta
00:00
Toque para gravar, ou responda em voz alta sem gravar
Perguntas de sabatina deste tema2
- · Descreva como uma mudança sua numa transformação chega ao repositório e ao ambiente das outras pessoas.
- · Você foi revisar a mudança de um colega num modelo que alimenta um indicador. O que você olha?