Skip to content

Commit 7c337da

Browse files
author
fabien.menager
committed
Rename methods to clarify purpose of default options creation
1 parent 99dec5e commit 7c337da

7 files changed

Lines changed: 11 additions & 8 deletions

File tree

src/PhenX.EntityFrameworkCore.BulkInsert.MySql/MySqlBulkInsertProvider.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ public MySqlBulkInsertProvider(ILogger<MySqlBulkInsertProvider>? logger = null)
2727
protected override string GetTempTableName(string tableName) => $"#_temp_bulk_insert_{tableName}";
2828

2929
/// <inheritdoc />
30-
protected override MySqlBulkInsertOptions GetDefaultOptions() => new();
30+
protected override MySqlBulkInsertOptions CreateDefaultOptions() => new();
3131

3232
/// <inheritdoc />
3333
public override Task<List<T>> BulkInsertReturnEntities<T>(

src/PhenX.EntityFrameworkCore.BulkInsert.PostgreSql/PostgreSqlBulkInsertProvider.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ private static string GetBinaryImportCommand(TableMetadata tableInfo, string tab
3333
}
3434

3535
/// <inheritdoc />
36-
protected override BulkInsertOptions GetDefaultOptions() => new()
36+
protected override BulkInsertOptions CreateDefaultOptions() => new()
3737
{
3838
BatchSize = 50_000,
3939
};

src/PhenX.EntityFrameworkCore.BulkInsert.SqlServer/SqlServerBulkInsertProvider.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ public SqlServerBulkInsertProvider(ILogger<SqlServerBulkInsertProvider>? logger
2828
/// <inheritdoc />
2929
protected override string GetTempTableName(string tableName) => $"#_temp_bulk_insert_{tableName}";
3030

31-
protected override SqlServerBulkInsertOptions GetDefaultOptions() => new()
31+
protected override SqlServerBulkInsertOptions CreateDefaultOptions() => new()
3232
{
3333
BatchSize = 50_000,
3434
};

src/PhenX.EntityFrameworkCore.BulkInsert.Sqlite/SqliteBulkInsertProvider.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ public SqliteBulkInsertProvider(ILogger<SqliteBulkInsertProvider>? logger = null
3030
protected override string AddTableCopyBulkInsertId => "--"; // No need to add an ID column in SQLite
3131

3232
/// <inheritdoc />
33-
protected override BulkInsertOptions GetDefaultOptions() => new()
33+
protected override BulkInsertOptions CreateDefaultOptions() => new()
3434
{
3535
BatchSize = 5,
3636
};

src/PhenX.EntityFrameworkCore.BulkInsert/Abstractions/IBulkInsertProvider.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,5 +42,5 @@ internal Task BulkInsert<T>(
4242
/// <summary>
4343
/// Make the default options for the provider, can be a subclass of <see cref="BulkInsertOptions"/>.
4444
/// </summary>
45-
internal BulkInsertOptions InternalGetDefaultOptions();
45+
internal BulkInsertOptions InternalCreateDefaultOptions();
4646
}

src/PhenX.EntityFrameworkCore.BulkInsert/BulkInsertProviderBase.cs

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -259,9 +259,12 @@ public virtual async Task BulkInsert<T>(
259259
}
260260
}
261261

262-
public BulkInsertOptions InternalGetDefaultOptions() => GetDefaultOptions();
262+
public BulkInsertOptions InternalCreateDefaultOptions() => CreateDefaultOptions();
263263

264-
protected abstract TOptions GetDefaultOptions();
264+
/// <summary>
265+
/// Create the default options for the provider, can be a subclass of <see cref="BulkInsertOptions"/>.
266+
/// </summary>
267+
protected abstract TOptions CreateDefaultOptions();
265268

266269
private async Task<(string TableName, DbConnection Connection)> PerformBulkInsertAsync<T>(
267270
bool sync,

src/PhenX.EntityFrameworkCore.BulkInsert/Extensions/DbSetExtensions.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -246,7 +246,7 @@ private static IBulkInsertProvider InitProvider<T, TOptions>(DbSet<T> dbSet, Act
246246
context = dbSet.GetDbContext();
247247
var provider = context.GetService<IBulkInsertProvider>();
248248

249-
var defaultOptions = provider.InternalGetDefaultOptions();
249+
var defaultOptions = provider.InternalCreateDefaultOptions();
250250

251251
if (defaultOptions is not TOptions castedOptions)
252252
{

0 commit comments

Comments
 (0)