Proposal for Redis cache tags using sets

[binaryfire]

Hello. I suggest implementing cache tags using Redis sets, similar to how it's done in Laravel. It's a more efficient way to retrieve a group of keys than using scan. What do you think about this approach? I can start working on a PR if everyone agrees.

**/cache/src/Driver/RedisDriver.php**

public function set($key, $value, $ttl = null, $tag = null): bool
{
    $seconds = $this->secondsUntil($ttl);
    $res = $this->packer->pack($value);
    
    if ($seconds > 0) {
        $setResult = $this->redis->set($this->getCacheKey($key), $res, $seconds);
    } else {
        $setResult = $this->redis->set($this->getCacheKey($key), $res);
    }
    
    if ($tag && $setResult) {
        // Add key to tag set
        $this->redis->sAdd("tag:{$tag}", $this->getCacheKey($key));
    }
    
    return $setResult;
}

public function deleteTag(string $tag): bool
{
    $members = $this->redis->smembers("tag:{$tag}");

    if ($members) {
        // Delete each member key
        $this->redis->del(...$members);
    }

    // Delete the set itself
    return $this->redis->del("tag:{$tag}") == 1;
}

However, to facilitate this, we would need to create a custom CacheInterface as the current PSR-16 CacheInterface doesn't provide a method for cache tagging. This would mean adding the $tag parameter to the set method and introducing a new deleteTag method.

[limingxinleo]

https://github.com/limingxinleo/i-cache

[binaryfire]

https://github.com/limingxinleo/i-cache

Only supports Hyperf v2

Also, wouldn't it be good to have this in the core hyperf/cache component? I think it would be simple to add.

[limingxinleo]

already support hyperf v3

I don't like this feature, so I wouldn't like to add it into hyperf/cache.