Github user zhitaoli commented on a diff in the pull request:

    https://github.com/apache/mesos/pull/283#discussion_r182226462
  
    --- Diff: src/slave/task_status_update_manager.cpp ---
    @@ -881,11 +881,21 @@ void TaskStatusUpdateStream::_handle(
         // Add it to the pending updates queue.
         pending.push(update);
       } else {
    -    // Record this ACK.
    -    acknowledged.insert(id::UUID::fromBytes(update.uuid()).get());
    +    StatusUpdate& pendingUpdate = pending.front();
    --- End diff --
    
    Make this const &


---

Reply via email to