* redis cache peer simple implementation, based on Memcached CachePeer

[crazedr0m]

Сабж )

[dovg]

Подозреваю, что в дифф попали изменения от предыдущего pull request

[crazedr0m]

есть немного ) по идее должны "уйти" после вмердживания watermark патча

[crazedr0m]

сделаю pull request заново

[crazedr0m]

Вот я не понял, почему в store() куча проверок, а в append мы сериализуем все подряд

Для него вообще не должно было работать. у меня и не работало.
$cache->set('a', 'a');
$this->assertEquals($cache->get('a'), 'a');
$cache->append('a', 'a');
$this->assertEquals($cache->get('a'), 'aa');

[AlexeyDsov]

а что в ассерте в этом примере получается?

[crazedr0m]

я решил доработать тесты, там еще кое-что выползает ) так что сначала доработаю тесты, а потом уже видно будет

[AlexeyDsov]

окей, подождем

[crazedr0m]

пока откладывается до фиксов остальных кеш пиров

[ents]

1. Давайте переименуем класс в RedisCache (потому что есть готовое расширениеhttps://github.com/nicolasff/phpredis, которое предоставляет класс redis) и будут конфликты имен

2. Почему бы не использовать готовый phpredis ? Разве не лучше сделать RedisCache как обертку над phpredis ? По аналогии с PeclMemcached

3. Возможно имеет смысл сделать отдельную ветку кешей - NoSQLCache extends CachePeer. Потому что у многих NoSQL решений есть дополнительные возможность (работа со списками, множествами, etc), и хорошо бы их объединить интерфейсом

[crazedr0m]

@ents 1. Давайте переименуем. Только там что-то мелькало про приставку Socket для Memcached ну и для этого заодно.
2. Можно использовать на здоровье. Обнаружил что уже существует как минимум 2 реализации только после того как дописал этот )
3. Наверное имеет смысл. Есть еще Tarantool Если я правильно понял, то кроме протокола мемкешед он еще и простой sql поддерживает. Вот куда его в БД или кеши? )

[ents]

@crazedr0m

1. Поддерживаю имя RedisSocket и аналогичное для Memcached
2. -
3. Используем трейты ? :))) А вообще здесь напрашивается решение сделать интерфейс IMap (ICache, IDictionary?), заимплементить его в CachePeer, и в Tarantool

[AlexeyDsov]

1. SocketMemcached вот тут - Code clean & add SocketMemcached instead of Memcached #78, думаю его правильней будет замержить после Sequence cache Sequence cache #89
2. Трейты использовать рановато.