Merge pull request #8 from FizzcodeSoftware/dev

Dev
FizzCodeSoftware · Jan 17, 2024 · fdea44d · fdea44d
2 parents e6becd8 + 2353dff
commit fdea44d
Show file tree

Hide file tree

Showing 58 changed files with 107 additions and 143 deletions.
diff --git a/.editorconfig b/.editorconfig
@@ -191,10 +191,22 @@ dotnet_diagnostic.RCS1194.severity = none
 # RCS1163: Unused parameter.
 dotnet_diagnostic.RCS1163.severity = silent
 
+# RCS1097: Remove redundant 'ToString' call
+dotnet_diagnostic.RCS1097.severity = silent
+
 # RCS1239: Use 'for' statement instead of 'while' statement.
 dotnet_diagnostic.RCS1239.severity = silent
 csharp_style_prefer_primary_constructors = true:suggestion
 
+# RCS1158: Static member in generic type should use a type parameter
+dotnet_diagnostic.RCS1158.severity = silent
+
+# RCS1202: Avoid NullReferenceException
+dotnet_diagnostic.RCS1202.severity = silent
+
+# RCS1075: Avoid empty catch clause that catches System.Exception
+dotnet_diagnostic.RCS1075.severity = silent
+
 [*.vb]
 # Modifier preferences
 visual_basic_preferred_modifier_order = Partial,Default,Private,Protected,Public,Friend,NotOverridable,Overridable,MustOverride,Overloads,Overrides,MustInherit,NotInheritable,Static,Shared,Shadows,ReadOnly,WriteOnly,Dim,Const,WithEvents,Widening,Narrowing,Custom,Async:suggestion
diff --git a/EtLast.AdoNet.MsSql/EtLast.AdoNet.MsSql.csproj b/EtLast.AdoNet.MsSql/EtLast.AdoNet.MsSql.csproj
@@ -17,8 +17,8 @@
 
   <ItemGroup>
     <PackageReference Include="FizzCode.LightWeight.AdoNet.DbConnection" Version="2.0.42.7056" />
-    <PackageReference Include="Microsoft.Data.SqlClient" Version="5.1.2" />
-    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.7.0">
+    <PackageReference Include="Microsoft.Data.SqlClient" Version="5.1.4" />
+    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.9.0">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>

diff --git a/EtLast.AdoNet.MsSql/Scopes/ResilientSqlScope/ResilientSqlScope.cs b/EtLast.AdoNet.MsSql/Scopes/ResilientSqlScope/ResilientSqlScope.cs
@@ -280,7 +280,6 @@ private class TableWithOrder
     }
 }
 
-
 [EditorBrowsable(EditorBrowsableState.Never)]
 public static class ResilientSqlScopeFluent
 {

diff --git a/EtLast.AdoNet.PostgreSql/EtLast.AdoNet.PostgreSql.csproj b/EtLast.AdoNet.PostgreSql/EtLast.AdoNet.PostgreSql.csproj
@@ -18,7 +18,7 @@
   <ItemGroup>
     <PackageReference Include="FizzCode.LightWeight.AdoNet.DbConnection" Version="2.0.42.7056" />
     <PackageReference Include="Npgsql" Version="8.0.1" />
-    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.7.0">
+    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.9.0">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>

diff --git a/EtLast.AdoNet/EtLast.AdoNet.csproj b/EtLast.AdoNet/EtLast.AdoNet.csproj
@@ -17,7 +17,7 @@
 
   <ItemGroup>
     <PackageReference Include="FizzCode.LightWeight.AdoNet.DbConnection" Version="2.0.42.7056" />
-    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.7.0">
+    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.9.0">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>

diff --git a/EtLast.AdoNet/EtlConnectionManager.cs b/EtLast.AdoNet/EtlConnectionManager.cs
@@ -129,10 +129,7 @@ public static void ReleaseConnection(IProcess process, ref DatabaseConnection co
             });
         },
         onClosed: connection => ioCommand.End(),
-        onError: (connection, ex) =>
-        {
-            ioCommand.Failed(ex);
-        });
+        onError: (connection, ex) => ioCommand.Failed(ex));
 
         if (connection == null)
             return;

diff --git a/EtLast.AdoNet/Mutators/MergeToSql/MergeToSqlMutator.cs b/EtLast.AdoNet/Mutators/MergeToSql/MergeToSqlMutator.cs
@@ -207,7 +207,6 @@ private void ExecuteStatements()
         _command = null;
         _statements.Clear();
     }
-
 }
 
 [EditorBrowsable(EditorBrowsableState.Never)]

diff --git a/EtLast.ConsoleHost/CommandImplementations/ModuleLoader.cs b/EtLast.ConsoleHost/CommandImplementations/ModuleLoader.cs
@@ -246,6 +246,7 @@ private static List<Type> FindTypesFromAppDomain<T>(string moduleName)
             {
                 var matchingTypes = assembly.GetTypes()
                     .Where(t => t.IsClass && !t.IsAbstract && interfaceType.IsAssignableFrom(t) && t.Namespace.EndsWith(moduleName, StringComparison.OrdinalIgnoreCase));
+
                 result.AddRange(matchingTypes);
             }
             catch (Exception) { }

diff --git a/EtLast.ConsoleHost/ConsoleHost.cs b/EtLast.ConsoleHost/ConsoleHost.cs
@@ -51,6 +51,8 @@ public ConsoleHost(string name)
         HostArgumentsFolder = @".\HostArguments";
         ReferenceAssemblyFolders.Add(@"C:\Program Files\dotnet\shared\Microsoft.NETCore.App\");
         ReferenceAssemblyFolders.Add(@"C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App\");
+
+        AppDomain.CurrentDomain.UnhandledException += UnhandledExceptionHandler;
     }
 
     protected override ILogger CreateHostLogger()

diff --git a/EtLast.ConsoleHost/ConsoleHostSessionBuilderExtensions.cs b/EtLast.ConsoleHost/ConsoleHostSessionBuilderExtensions.cs
@@ -3,7 +3,7 @@
 [EditorBrowsable(EditorBrowsableState.Never)]
 public static class ConsoleHostSessionBuilderExtensions
 {
-    public static ISessionBuilder UseRollingDevLogManifestFiles<T>(this ISessionBuilder session, int? maxFileCount, int? maxSizeOnDisk = 16 * 1024 * 1024)
+    public static ISessionBuilder UseRollingDevLogManifestFiles(this ISessionBuilder session, int? maxFileCount, int? maxSizeOnDisk = 16 * 1024 * 1024)
     {
         var folder = Path.Combine(session.DevLogFolder, "manifest");
         CleanupManifestFolder(maxFileCount, maxSizeOnDisk, folder);

diff --git a/EtLast.ConsoleHost/EtLast.ConsoleHost.csproj b/EtLast.ConsoleHost/EtLast.ConsoleHost.csproj
@@ -28,7 +28,7 @@
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>
     <PackageReference Include="Microsoft.CodeAnalysis.CSharp" Version="4.8.0" />
-    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.7.0">
+    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.9.0">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>

diff --git a/EtLast.ConsoleHost/Global.cs b/EtLast.ConsoleHost/Global.cs
@@ -7,16 +7,10 @@
 global using System.IO;
 global using System.Linq;
 global using System.Reflection;
-global using System.Runtime.InteropServices;
 global using System.Runtime.Loader;
 global using System.Text;
 global using System.Text.Json;
-global using System.Text.RegularExpressions;
 global using System.Threading;
 global using System.Transactions;
 global using Serilog;
-global using Serilog.Core;
 global using Serilog.Events;
-global using Serilog.Formatting.Compact;
-global using Serilog.Formatting.Display;
-global using Serilog.Parsing;
diff --git a/EtLast.Diagnostics.HttpSender/DiagnosticsHttpSender.cs b/EtLast.Diagnostics.HttpSender/DiagnosticsHttpSender.cs
@@ -21,7 +21,6 @@ public class DiagnosticsHttpSender : IDisposable, IEtlContextListener
     private readonly IEtlContext _context;
     private ExtendedBinaryWriter _currentWriter;
     private readonly ExtendedBinaryWriter _eventWriter = new(new MemoryStream(), Encoding.UTF8);
-    private readonly ExtendedBinaryWriter _dictWriter = new(new MemoryStream(), Encoding.UTF8);
     private readonly object _currentWriterLock = new();
     private bool _finished;
     private int _communicationErrorCount;

diff --git a/EtLast.Diagnostics.HttpSender/EtLast.Diagnostics.HttpSender.csproj b/EtLast.Diagnostics.HttpSender/EtLast.Diagnostics.HttpSender.csproj
@@ -12,7 +12,7 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.7.0">
+    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.9.0">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>

diff --git a/EtLast.Diagnostics.Interface/EtLast.Diagnostics.Interface.csproj b/EtLast.Diagnostics.Interface/EtLast.Diagnostics.Interface.csproj
@@ -12,7 +12,7 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.7.0">
+    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.9.0">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>

diff --git a/EtLast.Diagnostics.Windows/Controls/ContextProcessInvocationListControl.cs b/EtLast.Diagnostics.Windows/Controls/ContextProcessInvocationListControl.cs
@@ -175,14 +175,8 @@ private void ListView_MouseDoubleClick(object sender, MouseEventArgs e)
                     };
 
                     var control = new ProcessRowListControl(form, process, rows.Values.ToList());
-                    control.Updater.RefreshStarted += (sender, args) =>
-                    {
-                        form.Text = "LOADING...";
-                    };
-                    control.Updater.RefreshFinished += (sender, args) =>
-                    {
-                        form.Text = "Process output: " + process.Name;
-                    };
+                    control.Updater.RefreshStarted += (sender, args) => form.Text = "LOADING...";
+                    control.Updater.RefreshFinished += (sender, args) => form.Text = "Process output: " + process.Name;
 
                     ToolTipSingleton.Remove(ListView);
                     form.ShowDialog();

diff --git a/EtLast.Diagnostics.Windows/EtLast.Diagnostics.Windows.csproj b/EtLast.Diagnostics.Windows/EtLast.Diagnostics.Windows.csproj
@@ -19,7 +19,7 @@
 
   <ItemGroup>
     <PackageReference Include="ObjectListView.Repack.Core3" Version="2.9.3" />
-    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.7.0">
+    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.9.0">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>

diff --git a/EtLast.EPPlus/EtLast.EPPlus.csproj b/EtLast.EPPlus/EtLast.EPPlus.csproj
@@ -17,7 +17,7 @@
 
   <ItemGroup>
     <PackageReference Include="EPPlus" Version="4.5.3.3" />
-    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.7.0">
+    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.9.0">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>

diff --git a/EtLast.Hosting.AspNet/EtLast.Hosting.AspNet.csproj b/EtLast.Hosting.AspNet/EtLast.Hosting.AspNet.csproj
@@ -26,7 +26,7 @@
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>
     <PackageReference Include="Microsoft.CodeAnalysis.CSharp" Version="4.8.0" />
-    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.7.0">
+    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.9.0">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>

diff --git a/EtLast.Hosting/EtLast.Hosting.csproj b/EtLast.Hosting/EtLast.Hosting.csproj
@@ -16,7 +16,7 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.7.0">
+    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.9.0">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>

diff --git a/EtLast.Hosting/Host/AbstractHost.cs b/EtLast.Hosting/Host/AbstractHost.cs
@@ -199,7 +199,9 @@ private void UnhandledExceptionHandler(object sender, UnhandledExceptionEventArg
         var formattedMessage = ex.FormatExceptionWithDetails();
 
         if (Logger != null)
+        {
             Logger.Write(LogEventLevel.Fatal, "unexpected error during execution: {ErrorMessage}", formattedMessage);
+        }
         else
         {
             Console.WriteLine("unexpected error during execution: " + formattedMessage);

diff --git a/EtLast.Hosting/Logging/EtlContextDevToFileLogger.cs b/EtLast.Hosting/Logging/EtlContextDevToFileLogger.cs
@@ -12,39 +12,39 @@ public EtlContextDevToFileLogger(string folder, LogSeverity minimumLogLevel, int
         var config = new LoggerConfiguration()
             .WriteTo.File(new Serilog.Formatting.Compact.CompactJsonFormatter(), Path.Combine(folder, "events-.json"),
                 restrictedToMinimumLevel: (LogEventLevel)minimumLogLevel,
-                rollingInterval: RollingInterval.Day,
-                retainedFileCountLimit: infoFileCount,
                 buffered: true,
                 flushToDiskInterval: TimeSpan.FromSeconds(1),
+                rollingInterval: RollingInterval.Day,
+                retainedFileCountLimit: infoFileCount,
                 encoding: Encoding.UTF8)
 
             .WriteTo.File(Path.Combine(folder, "2-info-.txt"),
-                outputTemplate: "{Timestamp:HH:mm:ss.fff zzz} [{Level:u3}] {Message:l} {NewLine}{Exception}",
                 restrictedToMinimumLevel: LogEventLevel.Information,
+                outputTemplate: "{Timestamp:HH:mm:ss.fff zzz} [{Level:u3}] {Message:l} {NewLine}{Exception}",
                 formatProvider: CultureInfo.InvariantCulture,
                 rollingInterval: RollingInterval.Day,
                 retainedFileCountLimit: infoFileCount,
                 encoding: Encoding.UTF8)
 
             .WriteTo.File(Path.Combine(folder, "3-warning-.txt"),
-                outputTemplate: "{Timestamp:HH:mm:ss.fff zzz} [{Level:u3}] {Message:l} {NewLine}{Exception}",
                 restrictedToMinimumLevel: LogEventLevel.Warning,
+                outputTemplate: "{Timestamp:HH:mm:ss.fff zzz} [{Level:u3}] {Message:l} {NewLine}{Exception}",
                 formatProvider: CultureInfo.InvariantCulture,
                 rollingInterval: RollingInterval.Day,
                 retainedFileCountLimit: importantFileCount,
                 encoding: Encoding.UTF8)
 
             .WriteTo.File(Path.Combine(folder, "4-error-.txt"),
-                outputTemplate: "{Timestamp:HH:mm:ss.fff zzz} [{Level:u3}] {Message:l} {NewLine}{Exception}",
                 restrictedToMinimumLevel: LogEventLevel.Error,
+                outputTemplate: "{Timestamp:HH:mm:ss.fff zzz} [{Level:u3}] {Message:l} {NewLine}{Exception}",
                 formatProvider: CultureInfo.InvariantCulture,
                 rollingInterval: RollingInterval.Day,
                 retainedFileCountLimit: importantFileCount,
                 encoding: Encoding.UTF8)
 
             .WriteTo.File(Path.Combine(folder, "5-fatal-.txt"),
-                outputTemplate: "{Timestamp:HH:mm:ss.fff zzz} [{Level:u3}] {Message:l} {NewLine}{Exception}",
                 restrictedToMinimumLevel: LogEventLevel.Fatal,
+                outputTemplate: "{Timestamp:HH:mm:ss.fff zzz} [{Level:u3}] {Message:l} {NewLine}{Exception}",
                 formatProvider: CultureInfo.InvariantCulture,
                 rollingInterval: RollingInterval.Day,
                 retainedFileCountLimit: importantFileCount,
@@ -57,10 +57,10 @@ public EtlContextDevToFileLogger(string folder, LogSeverity minimumLogLevel, int
                     restrictedToMinimumLevel: LogEventLevel.Debug,
                     outputTemplate: "{Timestamp:HH:mm:ss.fff zzz} [{Level:u3}] {Message:l} {NewLine}{Exception}",
                     formatProvider: CultureInfo.InvariantCulture,
-                    rollingInterval: RollingInterval.Day,
-                    retainedFileCountLimit: lowFileCount,
                     buffered: true,
                     flushToDiskInterval: TimeSpan.FromSeconds(1),
+                    rollingInterval: RollingInterval.Day,
+                    retainedFileCountLimit: lowFileCount,
                     encoding: Encoding.UTF8);
         }
 
@@ -71,10 +71,10 @@ public EtlContextDevToFileLogger(string folder, LogSeverity minimumLogLevel, int
                     restrictedToMinimumLevel: LogEventLevel.Verbose,
                     outputTemplate: "{Timestamp:HH:mm:ss.fff zzz} [{Level:u3}] {Message:l} {NewLine}{Exception}",
                     formatProvider: CultureInfo.InvariantCulture,
-                    rollingInterval: RollingInterval.Day,
-                    retainedFileCountLimit: lowFileCount,
                     buffered: true,
                     flushToDiskInterval: TimeSpan.FromSeconds(1),
+                    rollingInterval: RollingInterval.Day,
+                    retainedFileCountLimit: lowFileCount,
                     encoding: Encoding.UTF8);
         }
 

diff --git a/EtLast.Hosting/Logging/EtlContextIoToFileLogger.cs b/EtLast.Hosting/Logging/EtlContextIoToFileLogger.cs
@@ -12,12 +12,12 @@ public EtlContextIoToFileLogger(string folder, int lowFileCount = 4)
             .WriteTo.File(Path.Combine(folder, "io-.tsv"),
                 outputTemplate: "{Timestamp:HH:mm:ss.fff zzz}\t{Message:l}{NewLine}",
                 formatProvider: CultureInfo.InvariantCulture,
-                rollingInterval: RollingInterval.Day,
-                retainedFileCountLimit: lowFileCount,
                 buffered: true,
                 flushToDiskInterval: TimeSpan.FromSeconds(1),
-                hooks: new IoFileLifecycleHooks(),
-                encoding: Encoding.UTF8);
+                rollingInterval: RollingInterval.Day,
+                retainedFileCountLimit: lowFileCount,
+                encoding: Encoding.UTF8,
+                hooks: new IoFileLifecycleHooks());
 
         _logger = config.CreateLogger();
     }
@@ -155,7 +155,6 @@ public override Stream OnFileOpened(string path, Stream underlyingStream, Encodi
                     writer.Flush();
                     underlyingStream.Flush();
                 }
-
             }
 
             return base.OnFileOpened(path, underlyingStream, encoding);

diff --git a/EtLast.LocalFiles/EtLast.LocalFiles.csproj b/EtLast.LocalFiles/EtLast.LocalFiles.csproj
@@ -16,7 +16,7 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.7.0">
+    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.9.0">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>

diff --git a/EtLast.ServiceModel/EtLast.ServiceModel.csproj b/EtLast.ServiceModel/EtLast.ServiceModel.csproj
@@ -17,7 +17,7 @@
 
   <ItemGroup>
     <PackageReference Include="System.ServiceModel.Primitives" Version="8.0.0" />
-    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.7.0">
+    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.9.0">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>

diff --git a/EtLast.Specialized/EtLast.Specialized.csproj b/EtLast.Specialized/EtLast.Specialized.csproj
@@ -17,7 +17,7 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.7.0">
+    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.9.0">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>

diff --git a/EtLast/EtLast.csproj b/EtLast/EtLast.csproj
@@ -16,7 +16,7 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.7.0">
+    <PackageReference Include="Roslynator.Formatting.Analyzers" Version="4.9.0">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>

diff --git a/EtLast/Mutators/Aggregation/Memory/MemoryAggregationMutator.cs b/EtLast/Mutators/Aggregation/Memory/MemoryAggregationMutator.cs
@@ -130,7 +130,7 @@ protected override IEnumerable<IRow> EvaluateImpl(Stopwatch netTimeStopwatch)
             }
 
             foreach (var row in groupRows)
-                (row as IRow).SetOwner(null);
+                (row as IRow)?.SetOwner(null);
 
             foreach (var aggregate in aggregates)
             {

diff --git a/EtLast/Mutators/Aggregation/Memory/SortedMemoryAggregationMutator.cs b/EtLast/Mutators/Aggregation/Memory/SortedMemoryAggregationMutator.cs
@@ -125,7 +125,7 @@ protected override IEnumerable<IRow> EvaluateImpl(Stopwatch netTimeStopwatch)
                     }
 
                     foreach (var groupRow in groupRows)
-                        (groupRow as IRow).SetOwner(null);
+                        (groupRow as IRow)?.SetOwner(null);
 
                     groupRows.Clear();
 
@@ -175,7 +175,7 @@ protected override IEnumerable<IRow> EvaluateImpl(Stopwatch netTimeStopwatch)
             }
 
             foreach (var groupRow in groupRows)
-                (groupRow as IRow).SetOwner(null);
+                (groupRow as IRow)?.SetOwner(null);
 
             groupRows.Clear();
 

diff --git a/EtLast/Mutators/ThrowExceptionOnRowErrorMutator.cs b/EtLast/Mutators/ThrowExceptionOnRowErrorMutator.cs
@@ -2,11 +2,6 @@
 
 public sealed class ThrowExceptionOnRowErrorMutator : AbstractMutator
 {
-    protected override void StartMutator()
-    {
-        base.StartMutator();
-    }
-
     protected override IEnumerable<IRow> MutateRow(IRow row, long rowInputIndex)
     {
         if (row.HasError())
-Original file line number
+Diff line change
@@ Expand Up / @@ -280,7 +280,6 @@ private class TableWithOrder @@
         }
     }
     [EditorBrowsable(EditorBrowsableState.Never)]
     public static class ResilientSqlScopeFluent
     {
@@ Expand Down @@