Re: Review Request 35697: mesos: Fixed test names.

2015-06-22 Thread Michael Park
> On June 22, 2015, 8:12 p.m., Ben Mahler wrote: > > Thanks, will commit this for you. > > > > Also we are inconcistent on the "Test" suffix, e.g. "MasterTest" vs > > "CRAMMD5Authentication". See two options: > > > > (1) We can use "Test" to signify that there is a fixture, if that's > > help

Re: Review Request 35697: mesos: Fixed test names.

2015-06-22 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35697/#review88840 --- Ship it! Thanks, will commit this for you. Also we are inconcisten

Re: Review Request 35697: mesos: Fixed test names.

2015-06-22 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35697/#review88808 --- Ship it! Thanks a bunch for these. - Till Toenshoff On June 20,

Re: Review Request 35697: mesos: Fixed test names.

2015-06-20 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35697/#review88682 --- Patch looks great! Reviews applied: [35697] All tests passed. - M

Review Request 35697: mesos: Fixed test names.

2015-06-20 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35697/ --- Review request for mesos, Benjamin Hindman and Till Toenshoff. Repository: meso