Merge pull request #466 from frankebersoll/patch-1

rasmus · web-flow · commit e718af55ec5a · 2018-05-23T20:51:57.000+02:00
Change Primary Key name for MSSQL Snapshot Store
diff --git a/RELEASE_NOTES.md b/RELEASE_NOTES.md
@@ -2,6 +2,8 @@
 
 * Fix: Commands are now correctly published when no events are emitted from a saga
   after handling a domain event
+* Minor fix: Updated name of Primary Key for MSSQL Snapshot Store to be different
+  from MSSQL Event Store, so both can be used in the same database without conflicts
 
 ### New in 0.58.3377 (released 2018-05-15)
 
diff --git a/Source/EventFlow.MsSql/SnapshotStores/Scripts/0001 - Create EventFlowSnapshots.sql b/Source/EventFlow.MsSql/SnapshotStores/Scripts/0001 - Create EventFlowSnapshots.sql
@@ -7,7 +7,7 @@ BEGIN
 		[AggregateSequenceNumber] [int] NOT NULL,
 		[Data] [nvarchar](MAX) NOT NULL,
 		[Metadata] [nvarchar](MAX) NOT NULL,
-		CONSTRAINT [PK_EventFlow] PRIMARY KEY CLUSTERED
+		CONSTRAINT [PK_EventFlowSnapshots] PRIMARY KEY CLUSTERED
 		(
 			[Id] ASC
 		)
@@ -19,4 +19,4 @@ BEGIN
 		[AggregateId] ASC,
 		[AggregateSequenceNumber] ASC
 	)
-END
+END

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ BEGIN`
`7`	`7`	`[AggregateSequenceNumber] [int] NOT NULL,`
`8`	`8`	`[Data] [nvarchar](MAX) NOT NULL,`
`9`	`9`	`[Metadata] [nvarchar](MAX) NOT NULL,`
`10`		`- CONSTRAINT [PK_EventFlow] PRIMARY KEY CLUSTERED`
	`10`	`+ CONSTRAINT [PK_EventFlowSnapshots] PRIMARY KEY CLUSTERED`
`11`	`11`	`(`
`12`	`12`	`[Id] ASC`
`13`	`13`	`)`
`@@ -19,4 +19,4 @@ BEGIN`
`19`	`19`	`[AggregateId] ASC,`
`20`	`20`	`[AggregateSequenceNumber] ASC`
`21`	`21`	`)`
`22`		`-END`
	`22`	`+END`