Remover múltiplas idas ao banco na consulta de contents

[filipedeschamps]

Contexto

A cada consulta de um Conteúdo, o Backend faz uma ida adicional ao Banco de Dados para calcular o número de filhos deste conteúdo e isto é feito nesta parte do código:

tabnews.com.br/models/content.js

Lines 51 to 64 in de65be9

	// TODO: this need to be optimized in the future.
	// Too many travels to the database just to get one value.
	for await (const contentObject of results.rows) {
	contentObject.children_deep_count = await findChildrenCount(
	{
	where: {
	id: contentObject.id,
	},
	},
	{
	transaction: options.transaction,
	}
	);
	}

Ou seja, se você pedir uma lista 30 conteúdos (que é o padrão da Home), é realizada mais 30 idas ao Banco de Dados para buscar a quantidade de respostas e injetar isso na propriedade children_deep_count.

Execução

Buscar todos esses valores em uma só ida ao Banco:

• perf(content): embed findChildrenCount() query inside findAll() method #672

[filipedeschamps]

Em Homologação o ganho foi muito maior (como esperado) e em Produção deu para notar um leve ganho de performance. Segue abaixo o comparativo para 30 itens:

Homologação (Antes)	Homologação (Depois)	Produção (Antes)	Produção (Depois)

E o comparativo para 100 itens (este eu esperava uma diferença maior):

Produção (antes)	Produção (depois)

Fechado por #672