Pular para o conteúdo

Tema

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 ver

O que uma boa mensagem de commit deve conter?

Quanta certeza você tem?

Vídeo · português · 31 min

Cobre o fluxo completo de contribuição e revisão, que é o que a pergunta cobra. Assista pensando em como isso se aplica a uma mudança de SQL, não de aplicação.

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

PassoPara que serve
Branch por mudançaIsolar o trabalho e permitir reverter só aquilo depois
Commits pequenos com intençãoExplicar o porquê, que o diff não mostra
Pedido de revisão com contextoDar ao revisor o que ele precisa para avaliar impacto
Verificação automáticaSintaxe, construção isolada, testes de dado
Revisão humanaGrão, regra de negócio, impacto, cobertura de teste
Integração e promoçãoMesmo 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:

  1. O que muda no número? Se a definição de um indicador mudou, isso é a informação mais importante do pedido.
  2. Quem consome o modelo afetado? Vem da linhagem, e define quem precisa ser avisado.
  3. 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 pegaO quê
FerramentaSintaxe, formatação, construção isolada, testes de dado, cobertura
PessoaGrã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
Complementar e prático, para quem nunca abriu um pedido de revisão. Repare no que ele escreve na descrição: é ali que o contexto da mudança vive.

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 rever

Drill

Automação ou pessoa?

Para cada item, diga quem deveria pegar isso.

  1. 1.Indentação e formatação do SQL.

  2. 2.A junção mudou o grão da tabela.

  3. 3.Coluna obrigatória ficou nula na saída.

  4. 4.A definição de cliente ativo mudou sem avisar a área de negócio.

  5. 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?
Responder no simulado
Para ir além3 artigos