Skip to content

Conversation

AlexeyDsov
Copy link
Member

В CarefulDatabaseRunner и UnifiedContainer сейчас используются begin/commit'ы.
Предлагаю заменить их на InnerTransaction. В случае

  • CarefulDatabaseRunner это позволит использовать класс уже внутри запущенно транзакции
  • UnifiedContainer упростит код, но при этом он перестанет проверять и использовать очередь запросов. Но подозреваю что вряд ли ее кто-то использует, т.к. с ней код получается громоздкий - см. убранные if'ы в UnifiedContainer'е.

Жду недельку замечаний/предложений, обсуждаем и мержу исключительно в master (1.1)

@AlexeyDsov
Copy link
Member Author

Вернемся к этому реквесту :)
Я наконец сделал то что хотел для Dialect'ов и DB и теперь SQLitePDO работает корректно с savepoint'ами. Думаю теперь можно его мержить, если ни от кого не последует возражений.

AlexeyDsov added a commit that referenced this pull request Oct 24, 2012
CarefulDatabaseRunner and UnifiedContainer will use InnerTransaction
@AlexeyDsov AlexeyDsov merged commit 3b60a2c into onPHP:master Oct 24, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant