Hello Justin, Thanks for reporting the minor issues. We will look into them and try fixing them by next release.
Thanks Amareshwari On Sat, May 2, 2015 at 7:03 AM, Justin Mclean <justinmcl...@me.com> wrote: > Hi, > > +1 binding > > > I did have issues getting the tests to run with this error, but I’d assume > it’s just my setup. > "There is no database named testqueryrewrite)” > You may want to add a little more information the README on running the > tests and what is required. You might also want to suggest recommended > values for MAVEN_OPTS. > > Binary LICENSE and NOTICE are comprehensive and seem correct, but I did't > check every detail. > > (Very) minor issues: > - MD5 content is oddly formatted (make comparison a little harder) > - Signed email has "(Works for Hadoop and ecosystem)” which is probably > not needed > - Pom.xml lists developers and their companies, software is developed by > individuals at the the ASF > > Thanks, > Justin > --------------------------------------------------------------------- > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org > For additional commands, e-mail: general-h...@incubator.apache.org > >
