daikon-cqrs / event-sourcing

Passed

Push — master ( 7f7df8...6d2aba )

by Thorsten

created 2017-07-08 00:00 UTC

Changes

	4 issues (4 major) were introduced. 1 issue was fixed.
	Test Coverage has decreased to 53% (-1%).
	10 added classes/operations
		A	ConcurrencyRaceLost added
		A	UnresolvableConflict added
		A	ConcurrencyRaceLost::__construct() added
		A	ConcurrencyRaceLost::getStreamId() added
		A	ConcurrencyRaceLost::getLostEvents() added
		A	UnitOfWork::getTrackedStream() added
		B	UnitOfWork::detectConflictingEvents() added
		A	UnresolvableConflict::__construct() added
		A	UnresolvableConflict::getStreamId() added
		A	UnresolvableConflict::getConflictingEvents() added
B	↗	A	Stream::appendEvents() improved

9.94

▶ ± 0

Test Coverage

53%

▼ -1% – Changes

Head	Thorsten Schmitt-Rink 2017-07-08 01:53 GMT+0200 first shot towards optimistic concurrency handling - move conflict detection from the Stream to the UOW, thereby separating data (Stream) from behavior (UOW) this will make it easier to move to a different strategy for conflict detection, e.g. a ConflictResolver being injected into the UOW as opposed to the current DomainEventInterface::conflictsWith approach - add two new Exceptions that indicate the type of error that occured during a commit attempt: racing the head and changesets that can not autoresolve - rename the StoreError to PossibleConflict disclaimer: newly introduced names are all subject to follow up discussions, refs #1
Base	Thorsten Schmitt-Rink 2017-07-07 22:13 UTC add contra point to the conflict detection comment of the Stream class