Merge branch 'release/9.1.1'

Author: AGiorgetti

.config/dotnet-tools.json

@@ -0,0 +1,12 @@
+{
+  "version": 1,
+  "isRoot": true,
+  "tools": {
+    "gitversion.tool": {
+      "version": "5.12.0",
+      "commands": [
+        "dotnet-gitversion"
+      ]
+    }
+  }
+}

Changelog.md

@@ -1,5 +1,17 @@
 # NEventStore.Persistence.Sql
 
+## 9.1.1
+
+- Target Framework supported: netstandard2.0, net462
+- Updated System.Data.SqlClient 4.8.5
+- Fix: NEventStore constraint failed with MySql 8.x (works with 5.7) [#487](https://github.com/NEventStore/NEventStore/issues/487)
+
+### Breaking Change
+
+- The fix for [#487](https://github.com/NEventStore/NEventStore/issues/487) changed how the `Commits` table is created for MySql 8.x:
+  to update an existing database in order to run on 8.x you need to manually update the `Commits` table schema and change the constraint of the `CommitId` column
+  from: `CommitId binary(16) NOT NULL CHECK (CommitId != 0)` to: `CommitId binary(16) NOT NULL CHECK (CommitId <> 0x00)`.
+
 ## 9.0.1 
 
 - Added documentation files to NuGet packages (improved intellisense support) [#36](https://github.com/NEventStore/NEventStore.Persistence.SQL/issues/36)

GitVersion.yml

@@ -1,56 +1,5 @@
-assembly-versioning-scheme: MajorMinorPatch
 mode: ContinuousDeployment
-tag-prefix: '[vV]'
-continuous-delivery-fallback-tag: ci
-major-version-bump-message: '\+semver:\s?(breaking|major)'
-minor-version-bump-message: '\+semver:\s?(feature|minor)'
-patch-version-bump-message: '\+semver:\s?(fix|patch)'
-legacy-semver-padding: 4
-build-metadata-padding: 4
-commit-message-incrementing: Enabled
-branches:
-  master:
-    mode: ContinuousDelivery
-    increment: Patch
-    prevent-increment-of-merged-branch-version: true
-    track-merge-target: false
-  release:
-    mode: ContinuousDeployment
-    tag: beta
-    increment: Patch
-    prevent-increment-of-merged-branch-version: true
-    track-merge-target: false
-  feature:
-    mode: ContinuousDeployment
-    tag: useBranchName
-    increment: Inherit
-    prevent-increment-of-merged-branch-version: false
-    track-merge-target: false
-  pull-requests:
-    regex: (pull|pull\-requests|pr)[/-]
-    mode: ContinuousDelivery
-    tag: PullRequest
-    increment: Inherit
-    prevent-increment-of-merged-branch-version: false
-    tag-number-pattern: '[/-](?<number>\d+)[-/]'
-    track-merge-target: false
-    source-branches: []
-  hotfix:
-    mode: ContinuousDeployment
-    tag: beta
-    increment: Patch
-    prevent-increment-of-merged-branch-version: false
-    track-merge-target: false
-  support:
-    mode: ContinuousDelivery
-    increment: Patch
-    prevent-increment-of-merged-branch-version: true
-    track-merge-target: false
-  develop:
-    mode: ContinuousDeployment
-    tag: unstable
-    increment: Minor
-    prevent-increment-of-merged-branch-version: false
-    track-merge-target: true
+branches: {}
 ignore:
   sha: []
+merge-message-formats: {}

README.md

@@ -5,10 +5,8 @@ SQL Persistence Engine for NEventStore
 
 NEventStore.Persistence.Sql currently supports:
 
-- .net framework 4.6.1
+- .net framework 4.6.2
 - .net standard 2.0
-- .net 5.0
-- .net 6.0
 - MsSql
 - SqlLite
 - MySql
@@ -53,10 +51,21 @@ git submodule update
 
 To build the project locally on a Windows Machine:
 
-- Install [Chocolatey](https://chocolatey.org/).
 - Optional: update `.\src\.nuget\NEventStore.Persistence.Sql.nuspec` file if needed (before creating relase packages).
 - Open a Powershell console in Administrative mode and run the build script `build.ps1` in the root of the repository.
 
+## How to Run Unit Tests (locally)
+
+- Install Database engines or use Docker to run them in a container (you can use the scripts in `./docker` folder).
+- Define the following environment variables:
+
+  ```
+  NEventStore.MsSql="Server=localhost,50001;Database=NEventStore;User Id=sa;Password=Password1;"
+  NEventStore.MySql="Server=localhost;Port=50003;Database=NEventStore;Uid=sa;Pwd=Password1;AutoEnlist=false;"
+  NEventStore.PostgreSql="Server=localhost;Port=50004;Database=NEventStore;Uid=sa;Pwd=Password1;Enlist=false;"
+  NEventStore.Oracle="Data Source=localhost:1521/XE;User Id=system;Password=Password1;Persist Security Info=True;"
+  ```
+
 ## How to contribute
 
 ### Git-Flow

appveyor.yml

@@ -88,7 +88,7 @@ test_script:
     iex "& $pgcmd"
 
 
-    nunit3-console "C:\projects\neventstore-persistence-sql\src\NEventStore.Persistence.MsSql.Tests\bin\Release\net461\NEventStore.Persistence.MsSql.Tests.dll" "C:\projects\neventstore-persistence-sql\src\NEventStore.Persistence.MySql.Tests\bin\Release\net461\NEventStore.Persistence.MySql.Tests.dll" "C:\projects\neventstore-persistence-sql\src\NEventStore.Persistence.PostgreSql.Tests\bin\Release\net461\NEventStore.Persistence.PostgreSql.Tests.dll" "C:\projects\neventstore-persistence-sql\src\NEventStore.Persistence.Sqlite.Tests\bin\Release\net461\NEventStore.Persistence.Sqlite.Tests.dll" --workers=0 --result=testsresults.xml;
+    nunit3-console "C:\projects\neventstore-persistence-sql\src\NEventStore.Persistence.MsSql.Tests\bin\Release\net462\NEventStore.Persistence.MsSql.Tests.dll" "C:\projects\neventstore-persistence-sql\src\NEventStore.Persistence.MySql.Tests\bin\Release\net462\NEventStore.Persistence.MySql.Tests.dll" "C:\projects\neventstore-persistence-sql\src\NEventStore.Persistence.PostgreSql.Tests\bin\Release\net462\NEventStore.Persistence.PostgreSql.Tests.dll" "C:\projects\neventstore-persistence-sql\src\NEventStore.Persistence.Sqlite.Tests\bin\Release\net462\NEventStore.Persistence.Sqlite.Tests.dll" --workers=0 --result=testsresults.xml;
 
 
     # upload results to AppVeyor

build.ps1

@@ -7,24 +7,21 @@ if ($configuration -eq '') {
 }
 $runtests = Read-Host 'Run Tests (y / n) [default:n] ?'
 
-# Consider using NuGet to download the package (GitVersion.CommandLine)
-choco install gitversion.portable --pre --y
-choco upgrade gitversion.portable --pre --y
-choco install nuget.commandline
-choco upgrade nuget.commandline
+# Install gitversion tool
+dotnet tool restore
 
 # Display minimal restore information
 dotnet restore ./src/NEventStore.Persistence.Sql.Core.sln --verbosity m
 
 # GitVersion (for the main module)
 Write-Host "Running GitVersion for the Project"
-$str = gitversion /updateAssemblyInfo | out-string
+$str = dotnet tool run dotnet-gitversion /updateAssemblyInfo | out-string
 $json = convertFrom-json $str
 $nugetversion = $json.NuGetVersion
 
 # Now we need to patch the AssemblyInfo for submodules
 Write-Host "Running GitVersion for the Dependencies"
-gitversion ".\dependencies\NEventStore" /updateAssemblyInfo | Out-Null
+dotnet tool run dotnet-gitversion ".\dependencies\NEventStore" /updateAssemblyInfo | Out-Null
 
 # Build
 Write-Host "Building: "$nugetversion" "$configuration

dependencies/NEventStore

@@ -32,7 +32,7 @@ services:
   mysql:
     container_name: nesci-mysql-1
     platform: linux
-    image: mysql:5.7
+    image: mysql:8.0 #5.7
     command: --default-authentication-plugin=mysql_native_password
     restart: always
     environment:

docker/docker-compose.ci.linux.db.yml

@@ -32,7 +32,7 @@ services:
   mysql:
     container_name: nesci-mysql-1
     platform: windows
-    image: mysql:5.7
+    image: mysql:8.0 #5.7
     command: --default-authentication-plugin=mysql_native_password
     restart: always
     environment:

docker/docker-compose.ci.windows.db.yml

@@ -20,31 +20,21 @@
 	6.x.x versions
 	-->
     <dependencies>
-      <group targetFramework=".NETFramework4.6.1">
-        <dependency id="NEventStore" version="[9.0.0,10.0.0)" exclude="Build,Analyzers"/>
-        <dependency id="Microsoft.Extensions.Logging.Abstractions" version="6.0.0" exclude="Build,Analyzers" />
-        <dependency id="System.Data.SqlClient" version="4.8.3" exclude="Build,Analyzers" />
-      </group>
-      <group targetFramework="net6.0">
-        <dependency id="NEventStore" version="[9.0.0, 10.0.0)" exclude="Build,Analyzers" />
-        <dependency id="Microsoft.Extensions.Logging.Abstractions" version="6.0.0" exclude="Build,Analyzers" />
-        <dependency id="System.Data.SqlClient" version="4.8.3" exclude="Build,Analyzers" />
-      </group>
-      <group targetFramework="net5.0">
-        <dependency id="NEventStore" version="[9.0.0, 10.0.0)" exclude="Build,Analyzers" />
-        <dependency id="Microsoft.Extensions.Logging.Abstractions" version="6.0.0" exclude="Build,Analyzers" />
+      <group targetFramework=".NETFramework4.6.2">
+        <dependency id="NEventStore" version="[9.1.1,10.0.0)" exclude="Build,Analyzers"/>
+        <dependency id="Microsoft.Extensions.Logging.Abstractions" version="7.0.1" exclude="Build,Analyzers" />
         <dependency id="System.Data.SqlClient" version="4.8.3" exclude="Build,Analyzers" />
       </group>
       <group targetFramework=".NETStandard2.0">
-        <dependency id="NEventStore" version="[9.0.0,10.0.0)" exclude="Build,Analyzers"/>
-        <dependency id="Microsoft.Extensions.Logging.Abstractions" version="6.0.0" exclude="Build,Analyzers" />
-        <dependency id="System.Data.SqlClient" version="4.8.3" exclude="Build,Analyzers" />
+        <dependency id="NEventStore" version="[9.1.1,10.0.0)" exclude="Build,Analyzers"/>
+        <dependency id="Microsoft.Extensions.Logging.Abstractions" version="7.0.1" exclude="Build,Analyzers" />
+        <dependency id="System.Data.SqlClient" version="4.8.5" exclude="Build,Analyzers" />
       </group>
     </dependencies>
     <frameworkAssemblies>
-      <frameworkAssembly assemblyName="System.Configuration" targetFramework=".NETFramework4.6.1" />
-      <frameworkAssembly assemblyName="System.Transactions" targetFramework=".NETFramework4.6.1" />
-      <frameworkAssembly assemblyName="System.Web" targetFramework=".NETFramework4.6.1" />
+      <frameworkAssembly assemblyName="System.Configuration" targetFramework=".NETFramework4.6.2" />
+      <frameworkAssembly assemblyName="System.Transactions" targetFramework=".NETFramework4.6.2" />
+      <frameworkAssembly assemblyName="System.Web" targetFramework=".NETFramework4.6.2" />
     </frameworkAssemblies>
   </metadata>
   <files>