Skip to content

Commit ddd6a2a

Browse files
committed
Address review comments
1 parent 0db00e8 commit ddd6a2a

File tree

3 files changed

+12
-12
lines changed

3 files changed

+12
-12
lines changed

src/EFCore.SqlServer/Extensions/SqlServerEntityTypeBuilderExtensions.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -330,7 +330,7 @@ public static bool CanSetIsTemporal(
330330
/// <param name="name">The name to assign to the index.</param>
331331
/// <returns>A builder to further configure the vector index.</returns>
332332
[Experimental(EFDiagnostics.SqlServerVectorSearch)]
333-
public static VectorIndexBuilder<TEntity> HasVectorIndex<TEntity>(
333+
public static SqlServerVectorIndexBuilder<TEntity> HasVectorIndex<TEntity>(
334334
this EntityTypeBuilder<TEntity> entityTypeBuilder,
335335
Expression<Func<TEntity, object?>> indexExpression,
336336
string metric,
@@ -345,7 +345,7 @@ public static VectorIndexBuilder<TEntity> HasVectorIndex<TEntity>(
345345
: entityTypeBuilder.HasIndex(indexExpression, name);
346346
indexBuilder.Metadata.SetVectorMetric(metric);
347347

348-
return new VectorIndexBuilder<TEntity>(indexBuilder);
348+
return new SqlServerVectorIndexBuilder<TEntity>(indexBuilder);
349349
}
350350

351351
/// <summary>
@@ -381,7 +381,7 @@ public static VectorIndexBuilder<TEntity> HasVectorIndex<TEntity>(
381381
/// <param name="name">The name to assign to the index.</param>
382382
/// <returns>A builder to further configure the vector index.</returns>
383383
[Experimental(EFDiagnostics.SqlServerVectorSearch)]
384-
public static VectorIndexBuilder HasVectorIndex(
384+
public static SqlServerVectorIndexBuilder HasVectorIndex(
385385
this EntityTypeBuilder entityTypeBuilder,
386386
string propertyName,
387387
string metric,
@@ -395,7 +395,7 @@ public static VectorIndexBuilder HasVectorIndex(
395395
: entityTypeBuilder.HasIndex(propertyName, name);
396396
indexBuilder.Metadata.SetVectorMetric(metric);
397397

398-
return new VectorIndexBuilder(indexBuilder);
398+
return new SqlServerVectorIndexBuilder(indexBuilder);
399399
}
400400

401401
/// <summary>

src/EFCore.SqlServer/Metadata/Builders/VectorIndexBuilder.cs renamed to src/EFCore.SqlServer/Metadata/Builders/SqlServerVectorIndexBuilder.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ namespace Microsoft.EntityFrameworkCore.Metadata.Builders;
1515
/// </remarks>
1616
/// <param name="indexBuilder">The index builder.</param>
1717
[Experimental(EFDiagnostics.SqlServerVectorSearch)]
18-
public class VectorIndexBuilder(IndexBuilder indexBuilder)
18+
public class SqlServerVectorIndexBuilder(IndexBuilder indexBuilder)
1919
{
2020
/// <summary>
2121
/// The index being configured.
@@ -31,7 +31,7 @@ public virtual IMutableIndex Metadata
3131
/// </remarks>
3232
/// <param name="name">The name of the index.</param>
3333
/// <returns>A builder to further configure the index.</returns>
34-
public virtual VectorIndexBuilder HasDatabaseName(string? name)
34+
public virtual SqlServerVectorIndexBuilder HasDatabaseName(string? name)
3535
{
3636
Metadata.SetDatabaseName(name);
3737

@@ -48,7 +48,7 @@ public virtual VectorIndexBuilder HasDatabaseName(string? name)
4848
/// </remarks>
4949
/// <param name="metric">The similarity metric for the vector index (e.g. "cosine", "euclidean", "dot").</param>
5050
/// <returns>A builder to further configure the vector index.</returns>
51-
public virtual VectorIndexBuilder UseMetric(string metric)
51+
public virtual SqlServerVectorIndexBuilder UseMetric(string metric)
5252
{
5353
Check.NotEmpty(metric);
5454

@@ -67,7 +67,7 @@ public virtual VectorIndexBuilder UseMetric(string metric)
6767
/// </remarks>
6868
/// <param name="type">The type of the vector index (e.g. "DiskANN").</param>
6969
/// <returns>A builder to further configure the vector index.</returns>
70-
public virtual VectorIndexBuilder UseType(string? type)
70+
public virtual SqlServerVectorIndexBuilder UseType(string? type)
7171
{
7272
Metadata.SetVectorIndexType(type);
7373

src/EFCore.SqlServer/Metadata/Builders/VectorIndexBuilder`1.cs renamed to src/EFCore.SqlServer/Metadata/Builders/SqlServerVectorIndexBuilder`1.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ namespace Microsoft.EntityFrameworkCore.Metadata.Builders;
2020
/// </seealso>
2121
/// <seealso href="https://learn.microsoft.com/sql/relational-databases/vectors/vectors-sql-server">Vectors in the SQL Database Engine.</seealso>
2222
[Experimental(EFDiagnostics.SqlServerVectorSearch)]
23-
public class VectorIndexBuilder<TEntity>(IndexBuilder<TEntity> indexBuilder)
23+
public class SqlServerVectorIndexBuilder<TEntity>(IndexBuilder<TEntity> indexBuilder)
2424
{
2525
/// <summary>
2626
/// The index being configured.
@@ -36,7 +36,7 @@ public virtual IMutableIndex Metadata
3636
/// </remarks>
3737
/// <param name="name">The name of the index.</param>
3838
/// <returns>A builder to further configure the index.</returns>
39-
public virtual VectorIndexBuilder<TEntity> HasDatabaseName(string? name)
39+
public virtual SqlServerVectorIndexBuilder<TEntity> HasDatabaseName(string? name)
4040
{
4141
Metadata.SetDatabaseName(name);
4242

@@ -53,7 +53,7 @@ public virtual VectorIndexBuilder<TEntity> HasDatabaseName(string? name)
5353
/// </remarks>
5454
/// <param name="metric">The similarity metric for the vector index (e.g. "cosine", "euclidean", "dot").</param>
5555
/// <returns>A builder to further configure the vector index.</returns>
56-
public virtual VectorIndexBuilder<TEntity> UseMetric(string metric)
56+
public virtual SqlServerVectorIndexBuilder<TEntity> UseMetric(string metric)
5757
{
5858
Check.NotEmpty(metric);
5959

@@ -72,7 +72,7 @@ public virtual VectorIndexBuilder<TEntity> UseMetric(string metric)
7272
/// </remarks>
7373
/// <param name="type">The type of the vector index (e.g. "DiskANN").</param>
7474
/// <returns>A builder to further configure the vector index.</returns>
75-
public virtual VectorIndexBuilder<TEntity> UseType(string? type)
75+
public virtual SqlServerVectorIndexBuilder<TEntity> UseType(string? type)
7676
{
7777
Metadata.SetVectorIndexType(type);
7878

0 commit comments

Comments
 (0)