This is an automated email from the ASF dual-hosted git repository.

mzhu pushed a commit to branch 1.8.x
in repository https://gitbox.apache.org/repos/asf/mesos.git


The following commit(s) were added to refs/heads/1.8.x by this push:
     new 855d1e7  Fixed a performance issue in the random sorter.
855d1e7 is described below

commit 855d1e79a401828176fe36b2cc1182d6856817b0
Author: Meng Zhu <[email protected]>
AuthorDate: Sun Apr 28 15:53:17 2019 -0700

    Fixed a performance issue in the random sorter.
    
    Review: https://reviews.apache.org/r/70564
---
 src/master/allocator/sorter/random/sorter.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/master/allocator/sorter/random/sorter.cpp 
b/src/master/allocator/sorter/random/sorter.cpp
index f4132bb..813f5b5 100644
--- a/src/master/allocator/sorter/random/sorter.cpp
+++ b/src/master/allocator/sorter/random/sorter.cpp
@@ -472,7 +472,7 @@ void RandomSorter::remove(const SlaveID& slaveId, const 
Resources& resources)
 vector<string> RandomSorter::sort()
 {
   pair<vector<string>, vector<double>> clientsAndWeights =
-    SortInfo(this).getClientsAndWeights();
+    sortInfo.getClientsAndWeights();
 
   weightedShuffle(
       clientsAndWeights.first.begin(),

Reply via email to