Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-09 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/#review105780 --- Bad patch! Reviews applied: [39832, 39882, 39839, 38579, 39015,

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-09 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/#review105659 --- Bad patch! Reviews applied: [38747] Failed command:

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-09 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/ --- (Updated Nov. 10, 2015, 1:25 a.m.) Review request for mesos and Timothy Chen.

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-08 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/ --- (Updated Nov. 9, 2015, 7:23 a.m.) Review request for mesos and Timothy Chen.

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-08 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/#review105658 --- Bad patch! Reviews applied: [38747] Failed command:

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-08 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/ --- (Updated Nov. 9, 2015, 7:55 a.m.) Review request for mesos and Timothy Chen.

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-06 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/ --- (Updated Nov. 7, 2015, 12:12 a.m.) Review request for mesos and Timothy Chen.

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-06 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/#review105565 --- Bad patch! Reviews applied: [38747] Failed command:

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-05 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/ --- (Updated Nov. 5, 2015, 4:01 p.m.) Review request for mesos and Timothy Chen.

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-05 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/#review105261 --- Bad patch! Reviews applied: [39838] Failed command:

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-05 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/ --- (Updated Nov. 5, 2015, 7:30 p.m.) Review request for mesos and Timothy Chen.

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-05 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/ --- (Updated Nov. 5, 2015, 7:33 p.m.) Review request for mesos and Timothy Chen.

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-05 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/#review105298 --- Bad patch! Reviews applied: [39832, 39882, 39839, 38579, 39015,

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-05 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/ --- (Updated Nov. 5, 2015, 11:59 p.m.) Review request for mesos and Timothy Chen.

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-05 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/#review105350 --- Bad patch! Reviews applied: [38747, 39250] Failed command:

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-04 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/#review105148 --- Bad patch! Reviews applied: [39838] Failed command:

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-04 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/ --- (Updated Nov. 4, 2015, 11:27 p.m.) Review request for mesos and Timothy Chen.

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-04 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/ --- (Updated Nov. 4, 2015, 6:09 p.m.) Review request for mesos and Timothy Chen.

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-01 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/#review104658 --- Bad patch! Reviews applied: [39838, 39832, 39839, 38579, 39015,

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-11-01 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/ --- (Updated Nov. 1, 2015, 3:39 p.m.) Review request for mesos and Timothy Chen.

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-10-15 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/#review102857 --- Patch looks great! Reviews applied: [39013, 38443, 39184, 39155,

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-10-15 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/ --- (Updated Oct. 15, 2015, 10:54 p.m.) Review request for mesos and Timothy Chen.

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-10-15 Thread Jojy Varghese
> On Oct. 15, 2015, 10:51 p.m., Timothy Chen wrote: > > Ship It! > > Timothy Chen wrote: > This doesn't seem to depend on anything from 38747. I'll be merging this > without the previous one. > > Timothy Chen wrote: > Btw please rebase this patch. The dependency is because

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-10-15 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/ --- (Updated Oct. 16, 2015, 12:17 a.m.) Review request for mesos and Timothy Chen.

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-10-15 Thread Timothy Chen
> On Oct. 15, 2015, 10:51 p.m., Timothy Chen wrote: > > Ship It! > > Timothy Chen wrote: > This doesn't seem to depend on anything from 38747. I'll be merging this > without the previous one. Btw please rebase this patch. - Timothy

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-10-15 Thread Timothy Chen
> On Oct. 15, 2015, 10:51 p.m., Timothy Chen wrote: > > Ship It! This doesn't seem to depend on anything from 38747. I'll be merging this without the previous one. - Timothy --- This is an automatically generated e-mail. To reply,

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-10-15 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/ --- (Updated Oct. 16, 2015, 12:20 a.m.) Review request for mesos and Timothy Chen.

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-10-15 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/#review102827 --- Ship it! Ship It!

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-10-14 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/#review102579 --- Patch looks great! Reviews applied: [39013, 38443, 39184, 39155,

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-10-13 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/ --- (Updated Oct. 13, 2015, 7:19 p.m.) Review request for mesos and Timothy Chen.

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-10-13 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/ --- (Updated Oct. 14, 2015, 12:45 a.m.) Review request for mesos and Timothy Chen.

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-10-12 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/#review102373 --- Bad patch! Reviews applied: [39013, 38443, 39184, 39155, 39156,

Re: Review Request 39250: Puller refactor: moved untar to a common place

2015-10-12 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/ --- (Updated Oct. 13, 2015, 12:37 a.m.) Review request for mesos and Timothy Chen.