facebookexperimental/Recoil

[Performance] Short-circuit set recoil state if the value is unchanged

Aberta

#1.927 aberto em 30 de jul. de 2022

Ver no GitHubGuia do colaborador
 (7 comentários) (1 reação) (0 responsável)JavaScript (1.151 forks)batch import
help wantedperformance

Métricas do repositório

Stars
 (19.428 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

It seems that the current implementation of setting recoil state can be further improved when the value is the same as the current value. In this case, we don't have to go through all the enchilada and more importantly don't have to call setState in the end, which is not exactly cost free. I have observed major performance hit thanks to that. I have to cache the previous value myself and do the equality check before calling set recoil state when recoil has all the knowledge to that.

Guia do colaborador

Direção de pesquisa
Examine a implementação atual do setRecoilState para identificar onde uma verificação de igualdade pode ser adicionada antes de prosseguir com a atualização de estado e a chamada setState.
Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
Performance
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
React
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70