Skip to content

Commit 8c72783

Browse files
authored
NodesManager cleanup (#8863)
1 parent d7c05c0 commit 8c72783

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

ydb/core/fq/libs/actors/nodes_manager.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,7 @@ class TNodesManagerActor : public NActors::TActorBootstrapped<TNodesManagerActor
145145
TPeer node = {SelfId().NodeId(), InstanceId + "," + HostName(), 0, 0, 0, DataCenter};
146146
bool selfPlacement = true;
147147
if (!Peers.empty()) {
148-
auto FirstPeer = NextPeer;
148+
auto firstPeer = NextPeer;
149149
while (true) {
150150
Y_ABORT_UNLESS(NextPeer < Peers.size());
151151
auto& nextNode = Peers[NextPeer];
@@ -154,9 +154,9 @@ class TNodesManagerActor : public NActors::TActorBootstrapped<TNodesManagerActor
154154
NextPeer = 0;
155155
}
156156

157-
if ( (!UseDataCenter || DataCenter.empty() || nextNode.DataCenter.empty() || DataCenter == nextNode.DataCenter) // non empty DC must match
158-
&& ( nextNode.MemoryLimit == 0 // memory is NOT limited
159-
|| nextNode.MemoryLimit >= nextNode.MemoryAllocated + totalMemoryLimit) // or enough
157+
if ((!UseDataCenter || DataCenter.empty() || nextNode.DataCenter.empty() || DataCenter == nextNode.DataCenter) // non empty DC must match
158+
&& (nextNode.MemoryLimit == 0 // memory is NOT limited
159+
|| nextNode.MemoryLimit >= nextNode.MemoryAllocated + totalMemoryLimit) // or enough
160160
) {
161161
// adjust allocated size to place next tasks correctly, will be reset after next health check
162162
nextNode.MemoryAllocated += totalMemoryLimit;
@@ -169,7 +169,7 @@ class TNodesManagerActor : public NActors::TActorBootstrapped<TNodesManagerActor
169169
break;
170170
}
171171

172-
if (NextPeer == FirstPeer) { // we closed loop w/o success, fallback to self placement then
172+
if (NextPeer == firstPeer) { // we closed loop w/o success, fallback to self placement then
173173
break;
174174
}
175175
}

0 commit comments

Comments
 (0)