Add indexes for SCIMRepresentationAttribute

Author: antifree

src/Scim/SimpleIdServer.Scim.MySQLMigrations/Migrations/20240706101356_AddIndexesForRepresentationAttribute.Designer.cs

@@ -0,0 +1,67 @@
+using Microsoft.EntityFrameworkCore.Metadata;
+using Microsoft.EntityFrameworkCore.Migrations;
+
+#nullable disable
+
+namespace SimpleIdServer.Scim.MySQLMigrations.Migrations
+{
+    /// <inheritdoc />
+    public partial class AddIndexesForRepresentationAttribute : Migration
+    {
+        /// <inheritdoc />
+        protected override void Up(MigrationBuilder migrationBuilder)
+        {
+            migrationBuilder.AlterColumn<string>(
+                name: "FullPath",
+                schema: "dbo",
+                table: "SCIMRepresentationAttributeLst",
+                type: "varchar(512)",
+                maxLength: 512,
+                nullable: true,
+                oldClrType: typeof(string),
+                oldType: "longtext",
+                oldNullable: true)
+                .Annotation("MySql:CharSet", "utf8mb4")
+                .OldAnnotation("MySql:CharSet", "utf8mb4");
+
+            migrationBuilder.CreateIndex(
+                name: "IX_SCIMRepresentationAttributeLst_FullPath",
+                schema: "dbo",
+                table: "SCIMRepresentationAttributeLst",
+                column: "FullPath");
+
+            migrationBuilder.CreateIndex(
+                name: "IX_SCIMRepresentationAttributeLst_ValueString",
+                schema: "dbo",
+                table: "SCIMRepresentationAttributeLst",
+                column: "ValueString");
+        }
+
+        /// <inheritdoc />
+        protected override void Down(MigrationBuilder migrationBuilder)
+        {
+            migrationBuilder.DropIndex(
+                name: "IX_SCIMRepresentationAttributeLst_FullPath",
+                schema: "dbo",
+                table: "SCIMRepresentationAttributeLst");
+
+            migrationBuilder.DropIndex(
+                name: "IX_SCIMRepresentationAttributeLst_ValueString",
+                schema: "dbo",
+                table: "SCIMRepresentationAttributeLst");
+
+            migrationBuilder.AlterColumn<string>(
+                name: "FullPath",
+                schema: "dbo",
+                table: "SCIMRepresentationAttributeLst",
+                type: "longtext",
+                nullable: true,
+                oldClrType: typeof(string),
+                oldType: "varchar(512)",
+                oldMaxLength: 512,
+                oldNullable: true)
+                .Annotation("MySql:CharSet", "utf8mb4")
+                .OldAnnotation("MySql:CharSet", "utf8mb4");
+        }
+    }
+}

src/Scim/SimpleIdServer.Scim.MySQLMigrations/Migrations/20240706101356_AddIndexesForRepresentationAttribute.cs

@@ -2,6 +2,7 @@
 using System;
 using Microsoft.EntityFrameworkCore;
 using Microsoft.EntityFrameworkCore.Infrastructure;
+using Microsoft.EntityFrameworkCore.Metadata;
 using Microsoft.EntityFrameworkCore.Storage.ValueConversion;
 using SimpleIdServer.Scim.Persistence.EF;
 
@@ -17,7 +18,7 @@ protected override void BuildModel(ModelBuilder modelBuilder)
 #pragma warning disable 612, 618
             modelBuilder
                 .HasDefaultSchema("dbo")
-                .HasAnnotation("ProductVersion", "7.0.18")
+                .HasAnnotation("ProductVersion", "8.0.4")
                 .HasAnnotation("Relational:MaxIdentifierLength", 64);
 
             modelBuilder.Entity("SCIMRepresentationSCIMSchema", b =>
@@ -212,7 +213,8 @@ protected override void BuildModel(ModelBuilder modelBuilder)
                         .HasColumnType("longtext");
 
                     b.Property<string>("FullPath")
-                        .HasColumnType("longtext");
+                        .HasMaxLength(512)
+                        .HasColumnType("varchar(512)");
 
                     b.Property<bool>("IsComputed")
                         .HasColumnType("tinyint(1)");
@@ -256,12 +258,16 @@ protected override void BuildModel(ModelBuilder modelBuilder)
 
                     b.HasKey("Id");
 
+                    b.HasIndex("FullPath");
+
                     b.HasIndex("ParentAttributeId");
 
                     b.HasIndex("RepresentationId");
 
                     b.HasIndex("SchemaAttributeId");
 
+                    b.HasIndex("ValueString");
+
                     b.ToTable("SCIMRepresentationAttributeLst", "dbo");
                 });
 

src/Scim/SimpleIdServer.Scim.MySQLMigrations/Migrations/SCIMDbContextModelSnapshot.cs

@@ -14,7 +14,11 @@ public void Configure(EntityTypeBuilder<SCIMRepresentationAttribute> builder)
             builder.HasOne(a => a.SchemaAttribute).WithMany().HasForeignKey(a => a.SchemaAttributeId).OnDelete(DeleteBehavior.Cascade);
             builder.HasMany(a => a.Children).WithOne().HasForeignKey("ParentAttributeId");
             builder.Ignore(a => a.CachedChildren);
+            builder.Property(a => a.FullPath).HasMaxLength(512);
             builder.Property(a => a.ValueString).HasMaxLength(255);
+
+            builder.HasIndex(a => a.FullPath);
+            builder.HasIndex(a => a.ValueString);
         }
     }
 }

src/Scim/SimpleIdServer.Scim.Persistence.EF/Configurations/SCIMRepresentationAttributeConfiguration.cs

@@ -11,7 +11,13 @@ public class SCIMDbContext : DbContext
     {
         private readonly SCIMEFOptions _options;
 
-        public SCIMDbContext(DbContextOptions<SCIMDbContext> dbContextOptions, IOptions<SCIMEFOptions> options) : base(dbContextOptions)
+        public SCIMDbContext(DbContextOptions<SCIMDbContext> dbContextOptions, IOptions<SCIMEFOptions> options): this((DbContextOptions)dbContextOptions, options)
+        {
+           
+        }
+
+        // protected constructor with generic options simplifies inheritance of the DbContext
+        protected SCIMDbContext(DbContextOptions dbContextOptions, IOptions<SCIMEFOptions> options): base(dbContextOptions)
         {
             _options = options.Value;
         }