Re: Review Request 46872: Updated quota.md and weights.md for set quota and update weight.

2016-06-17 Thread Guangya Liu
On 五月 2, 2016, 4:01 p.m., Guangya Liu wrote: > > Could you check whether this is an issue also with other endpoints? > > Guangya Liu wrote: > Thanks Joerg, others are using `JSON String` in the `curl` command and > only quota and weight need some update here. > > Alexander Rukletsov wrote:

Re: Review Request 46872: Updated quota.md and weights.md for set quota and update weight.

2016-06-17 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46872/#review138318 --- Ship it! Ship It! - Adam B On May 2, 2016, 3:35 p.m., Guangy

Re: Review Request 46872: Updated quota.md and weights.md for set quota and update weight.

2016-06-17 Thread Adam B
On May 2, 2016, 9:01 a.m., Guangya Liu wrote: > > Could you check whether this is an issue also with other endpoints? > > Guangya Liu wrote: > Thanks Joerg, others are using `JSON String` in the `curl` command and > only quota and weight need some update here. > > Alexander Rukletsov wrote

Re: Review Request 46872: Updated quota.md and weights.md for set quota and update weight.

2016-06-17 Thread Alexander Rukletsov
> On May 5, 2016, 6:14 a.m., Zhiwei Chen wrote: > > Here I think it's ok in document, since -d supports raw data and a file > > path. But this patch makes the doc more friendly to beginners. Indeed, both are fine. The only thing that concerns me is consistency across our examples; consistency

Re: Review Request 46872: Updated quota.md and weights.md for set quota and update weight.

2016-06-17 Thread Alexander Rukletsov
On May 2, 2016, 4:01 p.m., Guangya Liu wrote: > > Could you check whether this is an issue also with other endpoints? > > Guangya Liu wrote: > Thanks Joerg, others are using `JSON String` in the `curl` command and > only quota and weight need some update here. If others using JSON in examp

Re: Review Request 46872: Updated quota.md and weights.md for set quota and update weight.

2016-05-04 Thread Zhiwei Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46872/#review131789 --- Ship it! Here I think it's ok in document, since -d supports ra

Re: Review Request 46872: Updated quota.md and weights.md for set quota and update weight.

2016-05-02 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46872/#review131458 --- Patch looks great! Reviews applied: [46872] Passed command: expo

Re: Review Request 46872: Updated quota.md and weights.md for set quota and update weight.

2016-05-02 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46872/ --- (Updated 五月 2, 2016, 10:35 p.m.) Review request for mesos, Adam B, Alexander Ru

Re: Review Request 46872: Updated quota.md and weights.md for set quota and update weight.

2016-05-02 Thread Guangya Liu
On 五月 2, 2016, 4:01 p.m., Guangya Liu wrote: > > Could you check whether this is an issue also with other endpoints? Thanks Joerg, others are using `JSON String` in the `curl` command and only quota and weight need some update here. - Guangya

Re: Review Request 46872: Updated quota.md and weights.md for set quota and update weight.

2016-05-02 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46872/#review131319 --- docs/quota.md (line 87)

Re: Review Request 46872: Updated quota.md and weights.md for set quota and update weight.

2016-05-01 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46872/#review131263 --- Patch looks great! Reviews applied: [46872] Passed command: expo

Re: Review Request 46872: Updated quota.md and weights.md for set quota and update weight.

2016-04-30 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46872/ --- (Updated 五月 1, 2016, 6:42 a.m.) Review request for mesos, Adam B, Alexander Ruk