메모리 내 이벤트 소싱 알고리즘 - 맞습니까? 그것은 threadsafe입니까?

Question

이 질문의 약간의 투기성에 대한 사과.

시스템의 현재 상태를 유지하기 위해 메모리 내 정적 개체를 사용하여 Cms를 작성하고 이벤트 소싱을 사용하여 개체 상태를 다시 작성하고 롤백을 제공하는 데 사용할 수있는 이벤트 로그를 만듭니다. 등의 실행 또는 재 구축 시간이 오래 걸리는 경우 쌓아 쓰기의 많은의 가능성에서 제외

public class Cms 
{ 
    private static object WriteLock = new object(); 
    public static Cms Read { get; set; } 
    static Cms Write { get; set; } 

    static Cms() 
    { 
     Read = RebuildFromActionLog(); 
     Write = RebuildFromActionLog(); 
    } 

    public static void Update(Action action) 
    { 

     lock (WriteLock) 
     {     
      try 
      { 
       action.Apply(Write); 
      } 
      catch(Exception ex) 
      { 
       Write = RebuildFromActionLog(); //ditch the potentially messed up Write model 
       throw; 
      } 
      LogAction(action); //the action was a keeper, so keep it 
      Read = Write; //ditch the current read only model - it will continue to be used by any requests that have grabbed it 
      Write = RebuildFromActionLog(); //get a new model ready for the next write 
     } 
    } 
... 
} 

(더 내가 약에 누구인지 모를 경우 http://martinfowler.com/articles/lmax.html를 참조), 많은 메모리의 가능성 사용할 수있는 버그가 있습니까? 특히 동시성 관련 버그가 있습니까?

Answer 1

아무런 동기화가 없으므로 누군가가 공용 getter 또는 설정자인 Read을 사용하면 문제가 발생할 수 있습니다. 누군가 다른 스레드가 수정 한 것과 동시에 해당 특성을 사용하면 Update으로 읽음이 부실 값을 가져 오거나 쓰여진 변경 사항이 자동으로 없어 질 수 있습니다.

정말이 속성을 공개해야합니까? 공개적으로 쓰기 가능?