Re: Review Request 41763: Update a handful of dependencies.

2015-12-29 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41763/#review112181 --- Ship it! Ship It! - John Sirois On Dec. 29, 2015, 9:45 a.m., Bi

Re: Review Request 41763: Update a handful of dependencies.

2015-12-29 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41763/ --- (Updated Dec. 29, 2015, 8:45 a.m.) Review request for Aurora, Joshua Cohen and

Re: Review Request 41763: Update a handful of dependencies.

2015-12-29 Thread Bill Farner
> On Dec. 29, 2015, 8:38 a.m., Joshua Cohen wrote: > > mesos uses protobufs 2.5.0. Is there any danger in us using a newer > > version? Presumably 2.5 and 2.6 are compatible, but just want to be sure... I can't say with any authority, but their release notes don't mention anything about wire f

Re: Review Request 41763: Update a handful of dependencies.

2015-12-29 Thread Bill Farner
> On Dec. 29, 2015, 6:13 a.m., John Sirois wrote: > > build.gradle, line 108 > > > > > > I'm not positive what the rubric is, but it reads to me like all forces > > here above the '// Force ...' comment fold should

Re: Review Request 41763: Update a handful of dependencies.

2015-12-29 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41763/#review112177 --- Ship it! mesos uses protobufs 2.5.0. Is there any danger in us usi

Re: Review Request 41763: Update a handful of dependencies.

2015-12-29 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41763/#review112165 --- Ship it! build.gradle (line 108)

Re: Review Request 41763: Update a handful of dependencies.

2015-12-28 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41763/#review112118 --- Master (41c71d9) is green with this patch. ./build-support/jenkin