Hi Bill, Sorry didn't notice the formatting issue reviewBot was complaining about. I fixed it and seems green now.
On Wed, Jun 17, 2015 at 2:04 AM, Bill Farner <[email protected]> wrote: > Hey Mauricio! > > Thanks for the contribution! Please accept our apologies for the delay, > there are two reasons this review slipped through the cracks: > > 1.) our review bot replied indicating that the patch breaks the build. Due > to the review volume we deal with, reviewers tend to skim past reviews with > a red build. Please correct that issue and verify a green build locally. > > 2.) there are now reviewers on the 'People' line, and in practice we have > few people picking up the unassigned reviews. I'm one of those people, now > returning from vacation, so i've added myself as a reviewer :-) > > > > > -=Bill > > On Mon, Jun 15, 2015 at 8:21 AM, Mauricio Garavaglia < > [email protected]> wrote: > > > Hello, > > > > A couple of weeks ago I submitted a review that allows arbitrary > parameters > > to the docker container. Do you think this might be useful to be pushed > > upstream? Here's the link to the review entry > > https://reviews.apache.org/r/34337/ > > Thanks > > > > Mauricio > > >
