Skip to content

Commit e59b180

Browse files
fix linter and comments
1 parent 2f3bfe0 commit e59b180

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

slo/src/EF/SloTableContext.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ public class SloTableContext : SloTableContext<PooledDbContextFactory<TableDbCon
1212
protected override string Job => "EF";
1313

1414
protected override PooledDbContextFactory<TableDbContext> CreateClient(Config config) =>
15-
new(new DbContextOptionsBuilder<TableDbContext>().UseYdb(config.ConnectionString,
15+
new(new DbContextOptionsBuilder<TableDbContext>().UseYdb(config.ConnectionString,
1616
builder => builder.EnableRetryIdempotence()).Options);
1717

1818
protected override async Task Create(

src/EFCore.Ydb/src/Storage/Internal/YdbExecutionStrategy.cs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,19 +7,19 @@
77
namespace EntityFrameworkCore.Ydb.Storage.Internal;
88

99
/// <summary>
10-
/// Retry strategy for YDB.
10+
/// Retry strategy for YDB.<br/>
1111
///
12-
/// IMPORTANT:
13-
/// <br/>- The maximum number of attempts and backoff logic are encapsulated in <see cref="IRetryPolicy"/>.
12+
/// <br/>IMPORTANT:
13+
/// <br/>- The maximum number of attempts and backoff logic are encapsulated in <see cref="YdbRetryPolicy"/>.
1414
/// The base ExecutionStrategy parameters (maxRetryCount, maxRetryDelay) are not used.
1515
/// <br/>- This strategy must be invoked in the correct EF Core context/connection (YDB),
1616
/// so that exception types and ShouldRetryOn semantics match the provider.
17-
/// <br/>- This base <see cref="ExecutionStrategy"/> is a good place to emit metrics/logs (attempt number, delay, exception type, etc.).
17+
/// <br/>- The base <see cref="ExecutionStrategy"/> is a good place to emit metrics/logs (attempt number, delay, exception type, etc.).
1818
/// </summary>
1919
public class YdbExecutionStrategy(ExecutionStrategyDependencies dependencies, YdbRetryPolicyConfig retryPolicyConfig)
2020
// We pass "placeholders" to the base class:
21-
// - TimeSpan.Zero is not used in the real retry logic.
22-
// - Actual limits/delays are driven by IRetryPolicy.
21+
// - MaxAttempts and TimeSpan.Zero are not used in the real retry logic.
22+
// - Actual limits/delays are driven by YdbRetryPolicy.
2323
: ExecutionStrategy(dependencies, retryPolicyConfig.MaxAttempts, TimeSpan.Zero /* unused! */)
2424
{
2525
private readonly YdbRetryPolicy _retryPolicy = new(retryPolicyConfig);

0 commit comments

Comments
 (0)