Re: Review Request 67510: Fixed a bug in `createStrippedScalarQuantity()`.

2018-06-19 Thread Greg Mann

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67510/#review205008
---


Ship it!




Ship It!

- Greg Mann


On June 11, 2018, 5:30 p.m., Meng Zhu wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/67510/
> ---
> 
> (Updated June 11, 2018, 5:30 p.m.)
> 
> 
> Review request for mesos, Gastón Kleiman and Greg Mann.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> This patch fixes `createStrippedScalarQuantity()` by
> stripping the revocable field in resources.
> 
> Also added a test.
> 
> 
> Diffs
> -
> 
>   src/common/resources.cpp b1392012702865e39d17b8715747582529cbccca 
>   src/tests/resources_tests.cpp 2da583615ac8a8fb238f4996fe661343b718bad1 
>   src/v1/resources.cpp 50793271ea9b3013aa5b4dcd95f329db81dff4d8 
> 
> 
> Diff: https://reviews.apache.org/r/67510/diff/1/
> 
> 
> Testing
> ---
> 
> make check
> 
> 
> Thanks,
> 
> Meng Zhu
> 
>



Re: Review Request 67510: Fixed a bug in `createStrippedScalarQuantity()`.

2018-06-11 Thread Mesos Reviewbot Windows

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67510/#review204554
---



PASS: Mesos patch 67510 was successfully built and tested.

Reviews applied: `['67510']`

All the build artifacts available at: 
http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/67510

- Mesos Reviewbot Windows


On June 11, 2018, 5:30 p.m., Meng Zhu wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/67510/
> ---
> 
> (Updated June 11, 2018, 5:30 p.m.)
> 
> 
> Review request for mesos, Gastón Kleiman and Greg Mann.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> This patch fixes `createStrippedScalarQuantity()` by
> stripping the revocable field in resources.
> 
> Also added a test.
> 
> 
> Diffs
> -
> 
>   src/common/resources.cpp b1392012702865e39d17b8715747582529cbccca 
>   src/tests/resources_tests.cpp 2da583615ac8a8fb238f4996fe661343b718bad1 
>   src/v1/resources.cpp 50793271ea9b3013aa5b4dcd95f329db81dff4d8 
> 
> 
> Diff: https://reviews.apache.org/r/67510/diff/1/
> 
> 
> Testing
> ---
> 
> make check
> 
> 
> Thanks,
> 
> Meng Zhu
> 
>



Review Request 67510: Fixed a bug in `createStrippedScalarQuantity()`.

2018-06-11 Thread Meng Zhu

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67510/
---

Review request for mesos, Gastón Kleiman and Greg Mann.


Repository: mesos


Description
---

This patch fixes `createStrippedScalarQuantity()` by
stripping the revocable field in resources.

Also added a test.


Diffs
-

  src/common/resources.cpp b1392012702865e39d17b8715747582529cbccca 
  src/tests/resources_tests.cpp 2da583615ac8a8fb238f4996fe661343b718bad1 
  src/v1/resources.cpp 50793271ea9b3013aa5b4dcd95f329db81dff4d8 


Diff: https://reviews.apache.org/r/67510/diff/1/


Testing
---

make check


Thanks,

Meng Zhu