Skip to content

Commit

Permalink
fix: redis results store delete job
Browse files Browse the repository at this point in the history
  • Loading branch information
Lakshay Kalbhor committed Nov 30, 2023
1 parent ff4f99d commit 8a52f9d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions results/redis/results.go
Original file line number Diff line number Diff line change
Expand Up @@ -72,11 +72,11 @@ func New(o Options, lo *slog.Logger) *Results {

func (r *Results) DeleteJob(ctx context.Context, id string) error {
r.lo.Debug("deleting job")
if err := r.conn.LRem(ctx, resultPrefix+success, 1, id).Err(); err != nil {
if err := r.conn.ZRem(ctx, resultPrefix+success, 1, id).Err(); err != nil {
return err
}

if err := r.conn.LRem(ctx, resultPrefix+failed, 1, id).Err(); err != nil {
if err := r.conn.ZRem(ctx, resultPrefix+failed, 1, id).Err(); err != nil {
return err
}

Expand Down

0 comments on commit 8a52f9d

Please sign in to comment.