Skip to content

Commit

Permalink
fixes after merge
Browse files Browse the repository at this point in the history
  • Loading branch information
rekby committed Dec 15, 2023
1 parent c4a7fc7 commit 46ef8e8
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 2 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ on:
jobs:
autoformatter:
strategy:
fail-fast: false
matrix:
source-dir: ["./src/", "./examples/src/", "./slo/src/"]
include:
Expand Down Expand Up @@ -38,6 +39,7 @@ jobs:

inspection:
strategy:
fail-fast: false
matrix:
solutionPath: ["./src/YdbSdk.sln", "./examples/src/YdbExamples.sln", "./slo/src/src.sln"]
runs-on: ubuntu-latest
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ on:
jobs:
unit-tests:
strategy:
fail-fast: false
matrix:
os: [ubuntu-22.04, windows-2022, macos-12]
dotnet-version: [6.0.x, 7.0.x]
Expand Down
2 changes: 1 addition & 1 deletion examples/src/Common/DataUtils.cs
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ namespace Ydb.Sdk.Examples;

public record Series(ulong SeriesId, string Title, DateTime ReleaseDate, string Info)
{
public static Series FromRow(Sdk.Value.ResultSet.Row row)
public static Series FromRow(Value.ResultSet.Row row)
{
return new Series(
SeriesId: (ulong)row["series_id"].GetOptionalUint64()!,
Expand Down
1 change: 1 addition & 0 deletions slo/src/Client.cs
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
using Microsoft.Extensions.Logging;
using Microsoft.Extensions.Logging.Abstractions;
using Ydb.Sdk;
using Ydb.Sdk.Services.Sessions;
using Ydb.Sdk.Services.Table;

namespace slo;
Expand Down
1 change: 1 addition & 0 deletions src/Ydb.Sdk/src/Services/Sessions/SessionPoolBase.cs
Original file line number Diff line number Diff line change
Expand Up @@ -191,6 +191,7 @@ private void Dispose(bool disposing)
var task = DeleteSession(state.Session.Id);
tasks[i++] = task;
}

Task.WaitAll(tasks);
}

Expand Down
2 changes: 1 addition & 1 deletion src/Ydb.Sdk/src/Services/Table/ExecuteDataQuery.cs
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ public async Task<ExecuteDataQueryResponse> ExecuteDataQuery(
? TransactionState.Active
: TransactionState.Void;

tx = Transaction.FromProto(resultProto.TxMeta, _logger);
tx = Transaction.FromProto(resultProto.TxMeta, Logger);
}

ExecuteDataQueryResponse.ResultData? result = null;
Expand Down

0 comments on commit 46ef8e8

Please sign in to comment.