----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21404/#review42925 -----------------------------------------------------------
Ship it! Looks good to me. I think it was only used in the maven-plugin because that was created from a template. If it works without it, excellent! - Christopher Tubbs On May 13, 2014, 4:52 p.m., Sean Busbey wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21404/ > ----------------------------------------------------------- > > (Updated May 13, 2014, 4:52 p.m.) > > > Review request for accumulo. > > > Bugs: ACCUMULO-2804 > https://issues.apache.org/jira/browse/ACCUMULO-2804 > > > Repository: accumulo > > > Description > ------- > > We only use plexus-utils for two calls, each of which we can replace withs a > commons-* call. > > > Diffs > ----- > > maven-plugin/pom.xml da3b9136602fb309d7591147833aa742935f51bd > maven-plugin/src/main/java/org/apache/accumulo/maven/plugin/StartMojo.java > 414c08134066dc31b990447ef06eaf3c24614951 > pom.xml 5e32a555e35f97a00df236f16f9e0a2b12e0f0c9 > test/pom.xml 49dcd90c97fa932ab9e9d15ebf1ede007829eb43 > test/src/test/java/org/apache/accumulo/test/functional/MapReduceIT.java > 19462056520d841e1e1544e053b3eb857e47d6c8 > > Diff: https://reviews.apache.org/r/21404/diff/ > > > Testing > ------- > > unit tests, -P sunny ITs, MapReduceIT > > > Thanks, > > Sean Busbey > >
