IX_Messages_Position is redundant

[michael-freidgeim-webjet]

Microsoft Azure Performance shows recommendation to delete index:
Reason | Duplicate index
Duplicate index name | IX_Messages_Position
Original index name | PK_Events
Indeed https://github.com/SQLStreamStore/SQLStreamStore/blob/master/src/SqlStreamStore.MsSql/ScriptsV3/CreateSchema_v2.sql
has
CREATE TABLE dbo.Messages(
... CONSTRAINT PK_Events PRIMARY KEY NONCLUSTERED (Position),
and
CREATE UNIQUE NONCLUSTERED INDEX IX_Messages_Position ON dbo.Messages (Position);

So IX_Messages_Position should be removed from the script.

[damianh]

Thanks for the report. The schema for MsSqlV3 (in version 1.2) is locked. I'll look to incorporate this in next version.

[MNF]

It seems that the whole file https://github.com/SQLStreamStore/SQLStreamStore/blob/master/src/SqlStreamStore.MsSql/ScriptsV3/CreateSchema_v2.sql is not used anymore and can be deleted.