Skip to content

Commit

Permalink
Move savepoint creation back on the subscription store
Browse files Browse the repository at this point in the history
This reverts commit 194f387.

We discussed that we dont want to ensure exactly once delivery for external projections. Adding transaction logic via traits for the projections increases logic there. And while with alot of effort we could bring exactly once delivery to work for most cases php could still decide to die in the small timeframe where we commit the one transaction and then the second. So there is no gurantee except when using a dedicated store: neos/neos-development-collection#5377
  • Loading branch information
mhsdesign committed Dec 3, 2024
1 parent 2555f7d commit 59e2aaa
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions src/DoctrineDbalContentGraphProjection.php
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,6 @@
use Neos\ContentRepository\Core\Feature\WorkspaceRebase\Event\WorkspaceRebaseFailed;
use Neos\ContentRepository\Core\Feature\WorkspaceRebase\Event\WorkspaceWasRebased;
use Neos\ContentRepository\Core\Infrastructure\DbalSchemaDiff;
use Neos\ContentRepository\Core\Infrastructure\ProjectionTransactionTrait;
use Neos\ContentRepository\Core\NodeType\NodeTypeName;
use Neos\ContentRepository\Core\Projection\ContentGraph\ContentGraphProjectionInterface;
use Neos\ContentRepository\Core\Projection\ContentGraph\ContentGraphReadModelInterface;
Expand All @@ -81,8 +80,6 @@
*/
final class DoctrineDbalContentGraphProjection implements ContentGraphProjectionInterface
{
use ProjectionTransactionTrait;

use ContentStream;
use NodeMove;
use NodeRemoval;
Expand Down

0 comments on commit 59e2aaa

Please sign in to comment.