Twisted в действии — memcache на python

[VlK]

Интересная штучка. Вообще, Twisted — это та еще цаца.

Может, покажете еще и реализацию Entry?

[drJonnie]

Мне не совмем понятно, зачем вы сделали Cache синглтоном? Можно же было просто создавать его инстанс и передавать его MemcacheProtocol. В реальности это упрощает связи в программе. Да и потом, представьте, что вам потребуется заменить Cache на обычный dict. Сейчас это очень сложно, а с инстансом можно обойтись сабклассом: class DictCache(dict): ...

Очень режет глаз не указывание четких зависимостей в классах. Сейчас поясню. У вас Cache hard-coded в MemcacheProtocol. А хотелось бы видеть нечто подобное:

class MemcacheProtocol(LineOnlyReceiver):
    cache = Cache

# либо
class MemcacheProtocol(LineOnlyReceiver):
    def __init__(self, cache, ...):
        pass

Преимущество на лицо — когда надо заменить только вид кэша, просто создаем новый сабкласс или передаем другой кэш. К сожалению, даже в стандартной библиотеке такой подход редко встречается.

Жизненный пример: в модуле weakref есть WeakKeyDictionary. Когда потребовалось использовать свой конструктор для создания слабых связей, пришлось переписать полностью весь класс. А хотелось бы так:

class MyWeakDict(WeakKeyDictionary):
    weakref_factory = MyWeakref

Чем-то этот принцип напоминает inversion of control, только здесь я делаю упор просто на указание всех явных зависимостей класса, а не на их инициализацию.

[Deepwalker]

Большое спасибо! Я это обязательно кстати поменяю.

С Cache, хмм, ну я просто не совсем понимаю как лучше тут поступить — один и тот же его экземпляр должен использоваться пачкой экземпляров MemcacheProtocol, хотя тут я думаю можно экземпляр в фабрику записать. Точно, это гибче, ушел делать: )

[Deepwalker]

Закоммитил на github изменения, еще раз спасибо за очень дельное замечание. Действительно очень часто встречается ситуация, когда хочется что-то слегка подменить в поведении используемой библиотеки, а потом приходится тащить за собой ее исправленный код, из-за того, что встроенной возможности не было, и пришлось библиотеку править.

[drJonnie]

Вот-вот, я про то же.

[kmike]

А не поясните еще раз, зачем это? Академический интерес или что-то еще? Да, и у memcached фишка в распределенности, с этим тут как?

[Deepwalker]

Извините, я видимо не в теме, но про распределённость у memcached ничего не слышал. Из wikipedia:

Клиентская библиотека используя ключ данных вычисляет хэш и использует его для выбора соответствующего сервера. Ситуация сбоя сервера трактуется как промах кэша, что позволяет повышать отказоустойчивость комплекса за счет наращивания количества memcached серверов и возможности производить их горячую замену.

Если вы про эту распределённость, то можете втыкать этот сервер в один ряд с оригиналом — расширения, которые я сделал, совместимость не ломают.

Интерес прост — хочется иногда что-нибудь этакое сочинить. Ну и как вы могли заметить, я несколько расширил протокол. Если вы владеете python, то вам не составит труда вставить в сервер что-то свое.

[mkevac]

Распределенность реализуется не memcached, а его библиотеками — клиентами. Такими как libmemcached.

[plyushkin]

Зачем это, если у twisted уже есть реализация протокола memcached python.net/crew/mwh/apidocs/twisted.protocols.memcache.MemCacheProtocol.html

[VlK]

Часто бывает так, что надо поучиться.

И почему-то часто оказывается так, что все простые задачи уже сделаны :)

Что теперь, и таблицу умножения не учить? :-D

[plyushkin]

меня привлекает питон своей идеологией dont repeat yourself и batterie included
если питон использовать для учения, то желательно учитывать эту идеалогию ИМХО

[Deepwalker]

Иногда надо читать:

MemCache protocol: connect to a memcached server to store/retrieve values.