Re: Review Request 31951: Added test for long lived executors.

[Niklas Nielsen]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31951/
-----------------------------------------------------------

(Updated March 11, 2015, 2:17 p.m.)


Review request for mesos, Ben Mahler, Kapil Arya, and switched to 'mcypark'.


Changes
-------

Addressed Vinod's comments


Bugs: MESOS-2463
    https://issues.apache.org/jira/browse/MESOS-2463


Repository: mesos


Description
-------

Environment decorators broke long lived executors during master fail over 
(MESOS-2463).
This test verifies the fix suggested in r31889.


Diffs (updated)
-----

  src/tests/master_tests.cpp acf7050ce69a196ac74627d607b4664060fa1117 

Diff: https://reviews.apache.org/r/31951/diff/


Testing
-------

make check with and without r31889. (Breaks before with TASK_ERROR similar to 
MESOS-2463):

------------------------------------------------------------
Existing ExecutorInfo:
executor_id {
  value: "default"
}
command {
  environment {
  }
  value: "exit 1"
}
framework_id {
  value: "20150311-114156-3758096394-50894-29122-0000"
}

------------------------------------------------------------
Task's ExecutorInfo:
executor_id {
  value: "default"
}
command {
  value: "exit 1"
}
framework_id {
  value: "20150311-114156-3758096394-50894-29122-0000"
}

------------------------------------------------------------


Thanks,

Niklas Nielsen