CarefulDatabaseRunner & uncache objects

[zlex]

Есть проблема с анкешом при работе с CarefulDatabaseRunner: изменения объектов идут через dao, там же происходит анкеш объектов, но объекты в базе не меняются. После всех действий, если команда завершилась успешно, мы коммитим результат. После коммита мы не уверены, что в кеше не лежат старые объекты.

Вариант с повторным анкешом после коммита выглядит костыльным; здесь предлагаю обсудить варианты решения подобных кейсов.

[AlexeyDsov]

<?php
$command = new CarefulDatabaseRunner($innerCommand);
try {
    $command->run();
    $command->commit();
} catch (Exception $e) {
    //uncache operations
}

Ничего проще (т.е. более автоматического) сейчас в голову не приходит.

[zlex]

Под "если команда завершилась успешно" я имел в виду BaseEditor::COMMAND_SUCCEEDED:

<?php
$command = new CarefulDatabaseRunner($innerCommand);
$mav = $command->run();
if ($mav->getView() == BaseEditor::COMMAND_SUCCEEDED) {
    $command->commit();
    // objects really uncached?
}
?>

[AlexeyDsov]

Да, число подводных камней тут зашкаливает :)

[zlex]

Я вижу решение следующим образом:

1. Создать TransactionCache адаптер для Cache::me(). Накапливает модифицирующие события, по flush отправляет разом;
2. Научить воркеры lazy анкешу (возможно, наследник CacheDaoWorker) - отправляет запросы в адаптер TransactionCache;
3. Написать что-то вроде DaoTransaction по аналогии с DBTransaction. В начале транзакции выбираем воркер по заданному dao, указываем, что хотим lazy запросы в кеш. При коммите отправляем flush воркеру. Воркер отправляет flush в адаптер кешпира, меняет адаптер на текущий Cache::me();
4. CarefulDatabaseRunner использует DaoTransaction. Заодно уходим от deprecated DB::begin() в CarefulDatabaseRunner.

При этом если команда будет работать с несколькими dao, то возникают вопросы. Тут надо подумать.

В целом как-то много изменений получается для такого кейса... Критика приветствуется.

[AlexeyDsov]

Все еще хуже :D Не стоит забывать и об обратной хм... "связи".
Помимо раскэшивания после завершения транзакции есть еще беда что при изменении каких либо данных внутри транзакции нельзя уже верить кэшу, закешенному параллельными процессами.

К слову путь к отложенному раскэшиванию намечен, я просто еще не сделал большой pull request в котором содержится и эта тема. Можно глянуть у меня в fork'е: https://github.com/AlexeyDsov/onphp-framework/pull/2/files Там у воркеров добавился один метод. Я все еще раздумываю как оно правильно должно быть, но черновой вариант через анонимные функции можно уже сейчас увидеть.

[AlexeyDsov]

Гляньте следующие два коммита: Раз Два
По идее это получается именно то, что надо. Но я еще не тестил. Названия методов более чем обсуждаемы, т.к. муза меня покинула когда это быстрень набрасывал.

[AlexeyDsov]

Анкешеры попали в мастер. Бага должна перестать быть актуальной.