----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70887/#review216614 -----------------------------------------------------------
Fix it, then Ship it! src/common/future_track.hpp Lines 40 (patched) <https://reviews.apache.org/r/70887/#comment303825> Do you need the args to be ordered? If not, probably we could use `unordered_map` (or our stout hashmap wrapper)? the complexity is O(1) in average. src/common/future_track.hpp Lines 44-45 (patched) <https://reviews.apache.org/r/70887/#comment303826> nits: ``` return name == that.name && component == that.component && args == that.args; ``` src/common/future_track.hpp Lines 83 (patched) <https://reviews.apache.org/r/70887/#comment303840> For efficiency, if we are not expecting to erase frequently, we should use `vector` instead of `list`. in this class, the process will erase pending futures frequently? - Gilbert Song On June 19, 2019, 7:49 a.m., Andrei Budnik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/70887/ > ----------------------------------------------------------- > > (Updated June 19, 2019, 7:49 a.m.) > > > Review request for mesos, Benjamin Mahler, Gilbert Song, James Peach, Meng > Zhu, and Qian Zhang. > > > Bugs: MESOS-9837 > https://issues.apache.org/jira/browse/MESOS-9837 > > > Repository: mesos > > > Description > ------- > > This patch introduces a mechanism for tracking pending futures. > This feature allows detection of hanging operations, which get > stuck on a blocking operation or asynchronously. However, this > feature does not provide any mechanism for tracking pending > promises, because `Promise` objects might not be accessible in > various cases. Thereby, we introduce a new class that can be > used to track pending futures, so it might facilitate debugging > of stuck issues. > > > Diffs > ----- > > src/Makefile.am 761dde1d63e0f4f1ac4ab86f129f84f3746d3153 > src/common/future_tracker.hpp PRE-CREATION > > > Diff: https://reviews.apache.org/r/70887/diff/4/ > > > Testing > ------- > > > Thanks, > > Andrei Budnik > >