Skip to content

Commit c85af52

Browse files
committed
Fix review issues
1 parent 429746c commit c85af52

File tree

4 files changed

+8
-5
lines changed

4 files changed

+8
-5
lines changed

cloud/blockstore/libs/client/switchable_client.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -279,7 +279,7 @@ class TSwitchableBlockStore final
279279
return future.Apply(
280280
[sessionSwitcher = SessionSwitcher,
281281
diskId = PrimaryClientInfo.DiskId] //
282-
(TFuture<TResponse> future) -> TFuture<TResponse>
282+
(TFuture<TResponse> future) -> TResponse
283283
{
284284
TResponse response = future.ExtractValue();
285285

@@ -294,7 +294,7 @@ class TSwitchableBlockStore final
294294
}
295295
}
296296

297-
return MakeFuture(std::move(response));
297+
return response;
298298
});
299299
}
300300

@@ -310,7 +310,7 @@ class TSwitchableBlockStore final
310310
return future.Apply(
311311
[sessionSwitcher = SessionSwitcher] //
312312
(TFuture<NProto::TMountVolumeResponse> future)
313-
-> TFuture<NProto::TMountVolumeResponse>
313+
-> NProto::TMountVolumeResponse
314314
{
315315
NProto::TMountVolumeResponse response = future.ExtractValue();
316316

@@ -329,7 +329,7 @@ class TSwitchableBlockStore final
329329
}
330330
}
331331

332-
return MakeFuture(std::move(response));
332+
return response;
333333
});
334334
}
335335
};

cloud/blockstore/libs/client/switchable_client.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@ struct ISwitchableBlockStore: public IBlockStore
3333
};
3434

3535
////////////////////////////////////////////////////////////////////////////////
36+
3637
struct ISessionSwitcher
3738
{
3839
virtual ~ISessionSwitcher() = default;

cloud/blockstore/libs/client/switchable_client_ut.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -547,7 +547,7 @@ Y_UNIT_TEST_SUITE(TSwitchableClientTest)
547547
}
548548
}
549549

550-
Y_UNIT_TEST(ShouldSwitchSessionIfResponseContains_EF_OUTDATED_VOLUME)
550+
Y_UNIT_TEST(ShouldSwitchSessionIfResponseContainsFlagOutdatedVolume)
551551
{
552552
auto client1 = std::make_shared<TTestService>();
553553
auto sessionSwitcher = std::make_shared<TTestSessionSwitcher>();

cloud/blockstore/libs/client/ya.make

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
LIBRARY()
22

3+
INCLUDE(${ARCADIA_ROOT}/cloud/storage/deny_ydb_dependency.inc)
4+
35
SRCS(
46
client.cpp
57
config.cpp

0 commit comments

Comments
 (0)