> On July 12, 2015, 10:48 a.m., Alexander Rukletsov wrote:
> > src/master/http.cpp, lines 124-125
> > <https://reviews.apache.org/r/35687/diff/2/?file=1003411#file1003411line124>
> >
> > Hm, I wonder why didn't it work, did you get a compilation error? For example,
we have the following code in the master, which compiles fine:
> > ```
> > case Offer::Operation::LAUNCH: {
> > foreach (const TaskInfo& task, operation.launch().task_infos()) {
> > ...
> > ```
To @Vinod's comment above.
- Alexander
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35687/#review91408
-----------------------------------------------------------
On July 9, 2015, 4:04 p.m., Aditi Dixit wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/35687/
> -----------------------------------------------------------
>
> (Updated July 9, 2015, 4:04 p.m.)
>
>
> Review request for mesos, Marco Massenzio and Vinod Kone.
>
>
> Bugs: MESOS-2900
> https://issues.apache.org/jira/browse/MESOS-2900
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Added capabilities to state.json and test for the same
>
>
> Diffs
> -----
>
> src/master/http.cpp 350383362311cfbc830965e1155a8515f0dfb332
> src/tests/master_tests.cpp 962455cc368c6e5405599d6565660d4c3fd0fc22
>
> Diff: https://reviews.apache.org/r/35687/diff/
>
>
> Testing
> -------
>
> make check
>
>
> Thanks,
>
> Aditi Dixit
>
>
|