Re: Review Request 53825: MESOS-6597 Enabled java protos generation for all V1 proto files.

[Vijay Srinivasaraghavan <vi...@yahoo.com>]

> On Nov. 20, 2016, 9:04 p.m., Anand Mazumdar wrote:
> > Nice first patch and welcome to the community!
> > 
> > - Can you update the Testing Done section with details on testing?
> > - We also support Python bindings. Do you mind adding these protos to our python build too in a follow up review? (See my comments later on how you can use review dependencies if you are new to ReviewBoard)
> 
> Vijay Srinivasaraghavan wrote:
>     Thanks Anand. I have made the python binding changes locally on my machine. Do you want me to create a new JIRA bug for this issue and create a new review or add the patch to this JIRA/review itself? I am little confused as to how the dependencies are handled with RB.
> 
> Anand Mazumdar wrote:
>     No need to create a separate issue. It would be fine to create a new review with it only containing the python related changes. This would be another commit based on top of this one. You can then use `post-reviews.py` and it should handle setting the dependencies for you. Feel free to ping me on IRC/Slack if you run into any issues (anandm)
> 
> Vijay Srinivasaraghavan wrote:
>     Thanks for the clarification. I have attached a new patch with "53825" as dependendency

I have created two patch for those additional issues that was addressed as part of the original request. Please let me know if you are waiting for anything from my end before this can be merged? Thanks  
https://reviews.apache.org/r/54014/
https://reviews.apache.org/r/54015/


- Vijay


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


On Nov. 23, 2016, 5:46 a.m., Vijay Srinivasaraghavan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/53825/
> -----------------------------------------------------------
> 
> (Updated Nov. 23, 2016, 5:46 a.m.)
> 
> 
> Review request for mesos, Anand Mazumdar and Zameer Manji.
> 
> 
> Bugs: MESOS-6597
>     https://issues.apache.org/jira/browse/MESOS-6597
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> MESOS-6597 Enabled java protos generation for all V1 proto files.
> 
> 
> Diffs
> -----
> 
>   src/Makefile.am 5e0b8406f7f624bd8b03ff76b887f20e22fc66e0 
> 
> Diff: https://reviews.apache.org/r/53825/diff/
> 
> 
> Testing
> -------
> 
> Built mesos and confirmed jar file containing new java PB wrappers for V1 API. Ran standard unit test (make tests)
> 
> 
> Thanks,
> 
> Vijay Srinivasaraghavan
> 
>