mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From zhitaoli <...@git.apache.org>
Subject [GitHub] mesos pull request #283: Ignore acknowledge for task status update, received...
Date Tue, 17 Apr 2018 20:47:33 GMT
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 &


---

Mime
View raw message