You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by mz...@apache.org on 2019/04/30 17:39:59 UTC

[mesos] branch 1.8.x updated: Fixed a performance issue in the random sorter.

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 <mz...@mesosphere.io>
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(),