jazzband/django-redis

Support redis-py 4.0.2 Hash ops in django-redis

Open

#766 aberto em 13 de mar. de 2025

Ver no GitHub
 (0 comments) (2 reactions) (0 assignees)Python (452 forks)auto 404
good first issue

Métricas do repositório

Stars
 (3.075 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Problem Statement

django-redis doesn’t implement all the hash commands available in redis-py (4.0.2). I plan to work on implementing the missing commands, I’d appreciate any feedback on this proposal, especially regarding potential edge cases or test strategies.

Additional context

Currently supported hash commands

Hash Command django-redis redis-py 4.0.2 redis-py 5.2.0
HDEL V V V
HEXISTS V V V
HGET V V
HGETALL V V
HINCRBY V V
HINCRBYFLOAT V V
HKEYS V V V
HLEN V V V
HSET V V V
HSETNX V V
HMSET V V
HMGET V V
HVALS V V
HSTRLEN V V
HEXPIRE V
HPEXPIRE V
HEXPIREAT V
HPEXPIREAT V
HPERSIST V
HEXPIRETIME V
HPEXPIRETIME V
HTTL V
HPTTL V

related issue:

  • #736
  • #741
  • #373

Guia do colaborador

Direção de pesquisa
Estude as implementações existentes de comandos hash no django redis (por exemplo, HDEL, HEXISTS) para entender o padrão, depois implemente os comandos ausentes (HGET, HGETALL, etc.) seguindo esse padrão. Escreva os testes correspondentes.
Pilha de tecnologia
pythondjango
Domain
backend
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
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
GitPythonDjango
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80