Fix the loop of locking and releasing partitions (ydb-platform#8364)

nshestakov · web-flow · commit 1c40d489a81c · 2024-08-28T15:48:30.000+05:00
diff --git a/ydb/core/persqueue/read_balancer__balancing.cpp b/ydb/core/persqueue/read_balancer__balancing.cpp
@@ -1873,12 +1873,6 @@ bool LowLoadSessionComparator::operator()(const TSession* lhs, const TSession* r
     if (lhs->ActiveFamilyCount != rhs->ActiveFamilyCount) {
         return lhs->ActiveFamilyCount < rhs->ActiveFamilyCount;
     }
-    if (lhs->ActivePartitionCount != rhs->ActivePartitionCount) {
-        return lhs->ActivePartitionCount < rhs->ActivePartitionCount;
-    }
-    if (lhs->InactivePartitionCount != rhs->InactivePartitionCount) {
-        return lhs->InactivePartitionCount < rhs->InactivePartitionCount;
-    }
     if (lhs->Partitions.size() != rhs->Partitions.size()) {
         return lhs->Partitions.size() < rhs->Partitions.size();
     }

Original file line number	Diff line number	Diff line change
`@@ -1873,12 +1873,6 @@ bool LowLoadSessionComparator::operator()(const TSession* lhs, const TSession* r`
`1873`	`1873`	`if (lhs->ActiveFamilyCount != rhs->ActiveFamilyCount) {`
`1874`	`1874`	`return lhs->ActiveFamilyCount < rhs->ActiveFamilyCount;`
`1875`	`1875`	`}`
`1876`		`- if (lhs->ActivePartitionCount != rhs->ActivePartitionCount) {`
`1877`		`- return lhs->ActivePartitionCount < rhs->ActivePartitionCount;`
`1878`		`- }`
`1879`		`- if (lhs->InactivePartitionCount != rhs->InactivePartitionCount) {`
`1880`		`- return lhs->InactivePartitionCount < rhs->InactivePartitionCount;`
`1881`		`- }`
`1882`	`1876`	`if (lhs->Partitions.size() != rhs->Partitions.size()) {`
`1883`	`1877`	`return lhs->Partitions.size() < rhs->Partitions.size();`
`1884`	`1878`	`}`