Re: Review Request 36537: Made TaskState.data available via state.json endpoint.

2015-07-16 Thread Ben Mahler
> On July 16, 2015, 6:38 p.m., Ben Mahler wrote: > > We don't store 'data' because there are frameworks which send a lot of > > data, and this can OOM the master per MESOS-1746. Are you aware of this? > > Kapil Arya wrote: > Yes. That's why I haven't created the patch yet :-). I am still tr

Re: Review Request 36537: Made TaskState.data available via state.json endpoint.

2015-07-16 Thread Kapil Arya
> On July 16, 2015, 2:38 p.m., Ben Mahler wrote: > > We don't store 'data' because there are frameworks which send a lot of > > data, and this can OOM the master per MESOS-1746. Are you aware of this? Yes. That's why I haven't created the patch yet :-). I am still trying to explore avenues tha

Re: Review Request 36537: Made TaskState.data available via state.json endpoint.

2015-07-16 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36537/#review91937 --- We don't store 'data' because there are frameworks which send a lot

Re: Review Request 36537: Made TaskState.data available via state.json endpoint.

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

Re: Review Request 36537: Made TaskState.data available via state.json endpoint.

2015-07-16 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36537/ --- (Updated July 16, 2015, 10:54 a.m.) Review request for mesos, Benjamin Hindman