Yes, the PR is ready. As in the email that I sent yesterday, it's not the most elegant solution, but it seems to satisfy the requirements.
On Fri, Aug 19, 2016 at 6:54 AM, Tim Ellison <t.p.elli...@gmail.com> wrote: > On 18/08/16 18:53, smarthi wrote: > > Github user smarthi commented on the issue: > > > > https://github.com/apache/incubator-pirk/pull/65 > > > > +1 to merge, given that most of the feedback has been addressed now. > > Really? I have not done a thorough review, but a quick build of > remotes/ellisonanne/pirk-53 looked like there is still more to do. > > I'm waiting for Ellison Anne to declare when the PR is ready for further > scrutiny. > > Regards, > Tim >