You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mesos.apache.org by Jiang Yan Xu <ya...@jxu.me> on 2013/05/14 09:30:01 UTC

Review Request: Changed various *time* variables to use the "Time" type instead of "double".

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

Review request for mesos, Benjamin Hindman, Vinod Kone, and Ben Mahler.


Description
-------

See summary. [Part 12 of Duration-Time Refactoring Chain]


Diffs
-----

  src/master/http.cpp 6d496f5d69f1b5c87ff5de36970613bad6cb205b 
  src/master/master.hpp d3790dc2e8b2446736e8b7bcb39c5df54a791237 
  src/master/master.cpp 3207157961e772ae543a236536ae1559acfe110b 
  src/slave/http.cpp 32a742aba0326caf43076c542c8f8ce3a3834d18 
  src/slave/slave.hpp 82d2804446c658c4cde4d4fcfad2c8eaeec8b788 
  src/slave/slave.cpp d3126cc072cbf17cbc5cc18be4bfa4840886fb16 

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


Testing
-------

make check


Thanks,

Jiang Yan Xu

Re: Review Request: Changed various *time* variables to use the "Time" type instead of "double".

Posted by Jiang Yan Xu <ya...@jxu.me>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11148/
-----------------------------------------------------------

(Updated May 16, 2013, 8:28 a.m.)


Review request for mesos, Benjamin Hindman, Vinod Kone, and Ben Mahler.


Changes
-------

Captured two more uses of "double" time.


Description
-------

See summary. [Part 12 of Duration-Time Refactoring Chain]


Diffs (updated)
-----

  src/master/http.cpp 6d496f5d69f1b5c87ff5de36970613bad6cb205b 
  src/master/master.hpp d3790dc2e8b2446736e8b7bcb39c5df54a791237 
  src/master/master.cpp 3207157961e772ae543a236536ae1559acfe110b 
  src/slave/http.cpp 32a742aba0326caf43076c542c8f8ce3a3834d18 
  src/slave/slave.hpp 82d2804446c658c4cde4d4fcfad2c8eaeec8b788 
  src/slave/slave.cpp d3126cc072cbf17cbc5cc18be4bfa4840886fb16 

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


Testing
-------

make check


Thanks,

Jiang Yan Xu