> 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 > > Vijay Srinivasaraghavan wrote: > 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/ > > Anand Mazumdar wrote: > Sorry for the delay. I would get to them sometime later this week. There > is nothing needed from your end for now.
Thanks Anand - 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 > >
