Thread safety is too hard 😄

Author: cedx

src/Sql/Reflection/TableInfo.cs

@@ -1,6 +1,6 @@
 namespace Belin.Sql.Reflection;
 
-using System.Collections.Concurrent;
+using System.Collections.Frozen;
 using System.ComponentModel.DataAnnotations.Schema;
 using System.Reflection;
 
@@ -39,17 +39,14 @@ public sealed class TableInfo {
 	/// </summary>
 	/// <param name="type">The type information providing the table metadata.</param>
 	public TableInfo(Type type) {
-		var properties =
+		var table = type.GetCustomAttribute<TableAttribute>();
+		var columns =
 			from property in type.GetProperties(BindingFlags.Instance | BindingFlags.NonPublic | BindingFlags.Public)
 			where !property.IsDefined(typeof(NotMappedAttribute)) && ((property.CanRead && property.CanWrite) || property.IsDefined(typeof(ColumnAttribute)))
-			select property;
-
-		var table = type.GetCustomAttribute<TableAttribute>();
-		var columns = new ConcurrentDictionary<string, ColumnInfo>();
-		foreach (var property in properties) columns.TryAdd(property.Name, new ColumnInfo(property));
+			select new ColumnInfo(property);
 
-		Columns = columns;
-		IdentityColumn = columns.Values.SingleOrDefault(column => column.IsIdentity) ?? (columns.TryGetValue("Id", out var column) ? column : null);
+		Columns = columns.ToFrozenDictionary(column => column.Name);
+		IdentityColumn = Columns.Values.SingleOrDefault(column => column.IsIdentity) ?? (Columns.TryGetValue("Id", out var column) ? column : null);
 		Name = table?.Name ?? type.Name;
 		Schema = table?.Schema;
 		Type = type;

test/AssemblyInfo.cs

@@ -1 +1 @@
-[assembly: Parallelize(Scope = ExecutionScope.MethodLevel)]
+[assembly: Parallelize(Scope = ExecutionScope.ClassLevel)]