Addressed suggestion of reserve row indexes size

arangodb · neunhoef · Jul 17, 2022 · Jun 17, 2022 · Jun 20, 2022 · Jun 20, 2022
commit e5e4dc636b440777a748df36346e85c2cb01819f
diff --git a/arangod/Aql/SortedRowsStorageBackendMemory.cpp b/arangod/Aql/SortedRowsStorageBackendMemory.cpp
@@ -102,11 +102,9 @@ ExecutorState SortedRowsStorageBackendMemory::consumeInputRange(
 
   ResourceUsageScope guard(_infos.getResourceMonitor());
 
+
   if (_rowIndexes.capacity() < _rowIndexes.size() + numDataRows) {
-    size_t newCapacity = std::max(_rowIndexes.capacity() * 2, numDataRows);
-    while (newCapacity < _rowIndexes.size() + numDataRows) {
-      newCapacity *= 2;
-    }
+    size_t newCapacity = std::max(_rowIndexes.capacity() * 2, _rowIndexes.size() + numDataRows);
 
     // may throw
     guard.increase((newCapacity - _rowIndexes.capacity()) *
@@ -115,8 +113,6 @@ ExecutorState SortedRowsStorageBackendMemory::consumeInputRange(
     _rowIndexes.reserve(newCapacity);
   }
 
-  _rowIndexes.reserve(_rowIndexes.size() + numDataRows);
-
   InputAqlItemRow input{CreateInvalidInputRowHint{}};
 
   while (inputRange.hasDataRow()) {