Re: Review Request 72410: Updated existing documentation for resource limits.

2020-05-06 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72410/#review220669 --- Ship it! Ship It! - Qian Zhang On May 7, 2020, 8:35 a.m.,

Re: Review Request 72410: Updated existing documentation for resource limits.

2020-05-06 Thread Greg Mann
> On April 28, 2020, 9:59 a.m., Qian Zhang wrote: > > docs/task-state-reasons.md > > Lines 126-133 (patched) > > > > > > If we revert the changes about > > `REASON_CONTAINER_MEMORY_REQUEST_EXCEEDED` in > >

Re: Review Request 72410: Updated existing documentation for resource limits.

2020-05-06 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72410/ --- (Updated May 7, 2020, 12:35 a.m.) Review request for mesos, Andrei Budnik,

Re: Review Request 72410: Updated existing documentation for resource limits.

2020-04-28 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72410/#review220536 --- Patch looks great! Reviews applied: [72408, 72409, 72410]

Re: Review Request 72410: Updated existing documentation for resource limits.

2020-04-28 Thread Greg Mann
> On April 28, 2020, 9:59 a.m., Qian Zhang wrote: > > docs/task-state-reasons.md > > Lines 126-133 (patched) > > > > > > If we revert the changes about > > `REASON_CONTAINER_MEMORY_REQUEST_EXCEEDED` in > >

Re: Review Request 72410: Updated existing documentation for resource limits.

2020-04-28 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72410/#review220519 --- docs/task-state-reasons.md Lines 126-133 (patched)

Re: Review Request 72410: Updated existing documentation for resource limits.

2020-04-27 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72410/#review220511 --- Bad patch! Reviews applied: [72408, 72409, 72410] Failed

Re: Review Request 72410: Updated existing documentation for resource limits.

2020-04-27 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72410/ --- (Updated April 28, 2020, 12:59 a.m.) Review request for mesos, Andrei Budnik,

Re: Review Request 72410: Updated existing documentation for resource limits.

2020-04-27 Thread Greg Mann
> On April 22, 2020, 2:02 p.m., Qian Zhang wrote: > > docs/nested-container-and-task-group.md > > Lines 74 (patched) > > > > > > I see https://reviews.apache.org/r/72409 already has the content of > > this section,

Re: Review Request 72410: Updated existing documentation for resource limits.

2020-04-22 Thread Tim Harper
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72410/#review220430 --- Ship it! - Tim Harper On April 21, 2020, 8:01 p.m., Greg

Re: Review Request 72410: Updated existing documentation for resource limits.

2020-04-22 Thread Tim Harper
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72410/#review220429 --- docs/nested-container-and-task-group.md Line 65 (original), 65

Re: Review Request 72410: Updated existing documentation for resource limits.

2020-04-22 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72410/#review220423 --- docs/nested-container-and-task-group.md Lines 74 (patched)

Re: Review Request 72410: Updated existing documentation for resource limits.

2020-04-21 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72410/#review220407 --- Patch looks great! Reviews applied: [72408, 72409, 72410]

Review Request 72410: Updated existing documentation for resource limits.

2020-04-21 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72410/ --- Review request for mesos, Andrei Budnik, Benjamin Mahler, Qian Zhang, and Tim