Quero enfatizar que apoio tudo o que o macnator disse! Não sei o cargo do seu colega, porém se vocês estiverem lado a lado é importante você reportar esses casos para um superior. Desenvolver uma funcionalidade ou realizar uma correção não se trata apenas de adicionar o que é necessário e dane-se o resto. Você PODE e é EXTREMAMENTE NECESSÁRIO a manutenção do código, pois realmente isso pode vir a causar mais custos ou até mesmo churn de clientes para a empresa. Como você é um especialista e imagino que saiba bastante sobre performance, se agarre nisso e mostre que suas opiniões são válidas e podem sim ser aceitas.
Em resposta a Revisão de PR e relações pessoas - O dilema de PRs
1
1
Alem da questão da performance, existe tambem uma questão de manutenabilidade do codigo e testes que estão bastante escassos. Um dos motivos disso são as funcoes de 200, 300 linhas.
Performance acaba sendo algo mais pontual. Normalmente os desenvolvedores por aqui perdem muito mais tempo tentando entender o codigo e as regras de negocio do que qualquer outra coisa. Ja conversei com varios colegas desse time e de outros e é um problema geral na empresa.