O colega de trabalho renomeou todas as minhas consultas [fechadas]

63

Eu não sei se devo ficar muito irritado ou o quê. Eu sozinho construí mais de 300 consultas para um grande banco de dados, e desenvolvi uma convenção de nomes para que eu pudesse encontrá-los mais tarde. Ninguém mais no meu escritório sabe como construir uma consulta, mas cheguei ontem para descobrir que todos eles tinham sido renomeados. Agora estou tendo muita dificuldade em encontrar coisas e estou tentando descobrir o que fazer.

Falei com a pessoa responsável e ela minimizou a coisa toda. Ela disse que os renomeou para poder encontrá-los mais facilmente. Infelizmente, eu sou o único que sabe como construir, editar e mantê-los, e a única razão pela qual ela precisava encontrá-los era testar as consultas. A nova convenção de nomenclatura não faz sentido, e sinto que demos um passo para trás no processo de desenvolvimento.

O que estou tentando descobrir é:

1) Estou exagerando?

2) Qual é a melhor maneira de lidar com isso? Eu odeio mencionar isso ao meu chefe, mas depois de falar com minha colega de trabalho ontem, eu já posso dizer que ela sente que não fez nada errado.

    
por 6 revs, 3 users 60%anon 09.07.2011 / 21:48
fonte

15 respostas

81
  1. Não realmente - isso é uma coisa incrivelmente desrespeitosa.

  2. Você conversou com ela e ainda não o fez, mas parece que você tem o direito de restaurar as convenções de nomenclatura anteriores de um backup ou revertê-las se estiverem em um controle de origem. NÃO notifique seu chefe e seu colega de trabalho se você fizer isso e forneça sua razão (você não pode manter seu próprio trabalho).

A última coisa que você quer é entrar em uma frente e para trás sobre isso, embora lidar com isso como a situação parece warrent, mas deve pelo menos ser documentado no caso de se tornar parte de um padrão de desrespeito.

    
por 07.07.2011 / 14:31
fonte
117

Por que você simplesmente não lida com isso como adultos: sente-se, sem confrontos, e crie uma lista de prós e contras para um esquema de nomeação, concorde com um deles e torne-o oficial escrevendo um pequeno documento descrevendo-o . Elicite interesse genuíno em sua contribuição para que ela se sinta (e esteja) envolvida.

Se é principalmente uma questão de gosto e se ela é o tipo de pessoa que absolutamente tem que ter as coisas do seu jeito, então fique feliz que você é a pessoa maior e deixe-a ir. A vida é muito curta para ter um concurso de esquemas de nomeação.

O problema é o esquema de nomes ou que você sente que não recebe nenhum respeito? Se assim for, talvez você possa trabalhar em sua relação de trabalho. Se você acha que não vale a pena, por que você se importa com o que ela pensa? :) Outra opção pode ser que ela realmente não sente que é um grande negócio e se você explicar bem que você está tendo problemas para encontrar algo, talvez você possa mudá-lo de volta.

    
por 07.07.2011 / 17:48
fonte
36
  1. O design do banco de dados inclui permissões (GRANT e REVOKE).
  2. O teste inclui testes permissões.
  3. Relativamente poucas pessoas devem ter permissão para renomear banco de dados objetos.
  4. Seu colega de trabalho não é um dos poucos.
por 08.07.2011 / 03:05
fonte
21

"A nova convenção de nomenclatura não faz sentido", parece ser um desses:

  1. Ela aplicou alguma norma da empresa a eles. Eles costumam ser usados para garantir que o código seja pelo menos consistente e, na melhor das hipóteses, podem ajudar itens periféricos, como pequenos scripts personalizados, a localizar o código com facilidade. Nesse caso, você precisa entender a norma e por que ela "não faz sentido" em sua situação. Se você ainda acha que é melhor para todos os desenvolvedores deixá-lo como estava, explique a eles exatamente porque o seu método é superior, e pergunte se você poderia mudar a norma (provavelmente OK se eles concordarem que é superior ) ou renunciar ao seu caso (improvável e confuso a longo prazo).
  2. Ela inventou seu próprio padrão no local e aplicou isso. Isso é mais provável se ela é nova, e ela deve explicar sua justificativa. Você pode aprender alguma coisa, e / ou ela pode aprender alguma coisa se você explicar sua razão.

Um ponto importante é que não é seu código (singular), se ele pertence a alguma coisa e será modificado por todo o grupo. Nenhuma crítica sobre código deve estar centrada em torno de quem a escreveu.

    
por 07.07.2011 / 14:41
fonte
16

I spoke with the person responsible, and she just downplayed the whole thing.

Então, eu vou te dizer, sem a necessidade:

Roll back the changes.

Faça essa guerra. Seu gerente deve apoiá-lo e solidificar sua autoridade.

    
por 07.07.2011 / 18:52
fonte
10

1) Não, você não está reagindo demais. Alguém mudou seu trabalho sem lhe dizer e apagou quando você perguntou por quê. Isso é extremamente desrespeitoso e grosseiro.

2) Você é o DBA oficial, ou pelo menos a pessoa que foi criada como depositária do DB? Nesse caso, altere os nomes e escreva um documento de convenções sobre como você faz as coisas. Além disso, escreva um documento de estilo 'Guia do Usuário' para que, se alguém precisar entrar no banco de dados e encontrar algo que possa.

Eu enviaria isso para o grupo, sem apontar nenhum dedo, com uma nota útil de que você ficaria feliz em se sentar e fazer as pessoas passarem por algumas das nuances da estrutura.

Caso contrário, crie convenções em equipe e siga-as em equipe.

Em uma nota lateral, para alguém que teve que testar algo para alterar os nomes de mais de 300 consultas parece bastante infantil. Quanto tempo ela perdeu fazendo isso, e só para encontrar coisas? Em vez de simplesmente pedir ajuda a alguém, ela perdeu seu tempo, seu tempo e o tempo da empresa. Sem mencionar que o código provavelmente quebrou quando ela fez isso, assim desperdiçando o tempo de outro membro da equipe também.

Se eu fosse você, esperaria até você se acalmar, tente falar com ela novamente. Se isso não funcionar, use o chefe. Esse tipo de mentalidade cowboy vai colocar toda a equipe em um beco sem saída.

    
por 07.07.2011 / 14:47
fonte
8

Renomear aleatoriamente no banco de dados pode facilmente causar a queda do ambiente de produção. Se esses procedimentos fossem referenciados em algum lugar no código, isso poderia ter sérias conseqüências. Você pode fazer retrocessos, mas se um testador como esse realmente não sabe o que está fazendo, não é tão difícil ver o testador fazer algumas alterações na produção. Isso pode significar perda de negócios, e é por isso que você deve tentar implementar funções de usuário separadas para desenvolvedores e testadores. Nós fazemos isso com nossos testadores e funciona muito bem. Os testadores geralmente apreciam isso, porque não precisam viver com medo de estragar os dados ao vivo.

    
por 07.07.2011 / 22:47
fonte
5

Ele não parece ser mencionado em outro lugar, mas qualquer fonte (por exemplo, uma consulta) colocada em um local público deve estar sob um sistema de controle de versão.

Em seguida, se um colega de trabalho alterar seu esquema de nomenclatura, você poderá reverter facilmente para seu esquema de trabalho (e ver as alterações e, potencialmente, reverter, se necessário). Você também vincula as alterações a usuários específicos para ver quem estragou tudo.

    
por 07.07.2011 / 18:35
fonte
2

Não procure um presente na boca.

Em primeiro lugar, propriedade coletiva de código - eles não devem ser "seus".

Em segundo lugar, se eles os renomearem, pergunte o raciocínio em torno do novo esquema de nomenclatura. Ou eles estão usando as consultas - nesse caso, é o tipo de chamada deles; ou é um primeiro passo para eles começarem a te ajudar na manutenção deles.

Se todos acharem que são "seus", você nunca se livrará deles e seguirá em frente para algo novo.a

    
por 08.07.2011 / 08:03
fonte
2

Não sei se isso foi perguntado, mas qual convenção de nomenclatura é a versão oficial? Se a sua versão é oficial, então direi o problema da perspectiva. Então, em vez de dizer "Pessoa X revertida todas as minhas alterações", basta dizer "A pessoa X fez alterações que são contrárias às convenções de nomenclatura oficiais". Se não houver uma convenção oficial, sugiro que ela saiba que você não aprecia as alterações feitas sem consultá-lo primeiro.

Em ambos os casos, acho que travar uma "guerra" não é a resposta. Mesmo se você ganhar, você perde.

    
por 08.07.2011 / 15:55
fonte
1

Esse é um comportamento terrível. Parece que ela não se arrepende, então leve-o ao seu chefe e faça um caso para ter seu acesso revogado até que ela possa ser convencida a não mexer.

Se o seu chefe não é técnico, explique em termos que eles entenderão. Imagine começar a trabalhar em uma sala de correios, onde o correio é classificado em buracos de pombo prontos para entrega. Você decide unilateralmente ordenar os buracos de pombo pelo chão então o sobrenome em vez do sistema atual de departamento então chão. Isso pode facilitar sua vida a curto prazo, mas você seria assassinado pelos outros funcionários da sala de correios.

Está além de ser rude. Eu ficaria furioso.

    
por 08.07.2011 / 00:43
fonte
1

Além de definir permissões para impedir que pessoas aleatórias as alterem, você também deve explicar como é seu trabalho testar a funcionalidade; não é possível garantir qualquer confiabilidade se pessoas aleatórias estiverem fazendo alterações no código.

    
por 08.07.2011 / 00:50
fonte
1

Como todos disseram, ela não deveria ter feito isso, mesmo que apenas por respeito a você, já que você é o criador responsável por essas perguntas.

Tendo dito isso, eu não vejo ninguém mencionando o fato de que, se ela renomeou suas perguntas em primeiro lugar, foi porque ela não conseguia entender sua convenção de nomenclatura. Assim, o problema poderia ser facilmente resolvido documentando sua convenção de nomenclatura e garantindo que os colegas de trabalho tenham acesso ao documento e possam encontrar o que precisam.

Você também deve ser cuidadoso e levar em conta como outras pessoas encontrarão e usarão suas consultas: se sua convenção de nomenclatura não permitir que eles façam seu trabalho com eficiência, provavelmente será necessário manter uma lista mais completa de suas consultas. , usando talvez tags e palavras-chave acordadas para que outras pessoas possam encontrar o que estão procurando.

A chave aqui, eu acho, é que ninguém trabalha de forma isolada e a melhor maneira de evitar o pisoteio dos outros é comunicar e concordar sobre regras básicas comuns.

    
por 09.07.2011 / 05:26
fonte
0

Eu responderia da mesma maneira - minimizando sua decisão de reverter tudo de volta. Basta reverter as alterações e escrever um e-mail realmente curto para seus colegas de trabalho:

"Revertida alteração de rXXXX por enquanto, porque não entendi a convenção de nomenclatura. Obrigado por tentar. :)"

    
por 13.07.2011 / 22:48
fonte
0

Sim, você está exagerando.

Existe algo chamado controle de versão que, entre outras coisas, é usado para não ter que bater o $ #! 7 de colegas de trabalho quando eles mexem com suas coisas. Basta reverter para a versão anterior e bloquear o arquivo, deixando-a lidar com a raiva. Isso abrirá a oportunidade para você explicar que realizar mudanças radicais no código que tem dependências de outras pessoas sem uma razão sólida e sem perguntar primeiro não é apenas errado, extremamente impraticável e praticamente um pecado.

Claro que isso pressupõe que sua convenção de nomenclatura é melhor que a dela e que você pode realmente fazer backup dessa decisão com argumentos objetivos sólidos, se esse não for o caso, a melhor coisa a fazer é começar a alterar seu código assim que puder. as alterações e tente chegar a uma melhor convenção de nomenclatura da próxima vez.

Não leve isso ao seu chefe, a maneira madura de resolvê-lo é diretamente com o seu colega de trabalho, você terá que trabalhar com ela depois disso, então é estúpido prejudicar a relação para uma briga facilmente solucionável.

    
por 02.04.2012 / 18:54
fonte