----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15497/#review28973 -----------------------------------------------------------
Patch looks good to me. One thing to consider is that specifying the version of a dependent in a child pom is unnecessary if the dependent is specified in the parent dependent management. Specifying them in both places might create inconsistency in a long run. - Xuefu Zhang On Nov. 15, 2013, 5:24 p.m., Brock Noland wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15497/ > ----------------------------------------------------------- > > (Updated Nov. 15, 2013, 5:24 p.m.) > > > Review request for hive, Prasad Mujumdar and Thejas Nair. > > > Bugs: HIVE-5741 > https://issues.apache.org/jira/browse/HIVE-5741 > > > Repository: hive-git > > > Description > ------- > > OK the attached patches adds hcatalog to the binary tarball with mostly the > same structure as the 0.12 release. The delta is just that some jars have > moved slightly and there is significantly less jars in the webcat server lib > directory. Those two items will be addressed in followups. > > Additionally because the binary tarball lib directory was so dirty I have > used "dependency management" in the root pom to enforce our versions of jars > and upgraded some dependencies. > > > Diffs > ----- > > ant/pom.xml 6fdd375 > beeline/pom.xml 6c59229 > cli/pom.xml d5fba72 > common/pom.xml 0c62d9e > contrib/pom.xml 59a3cf9 > hbase-handler/pom.xml 64e7877 > hcatalog/core/pom.xml 358fa0a > hcatalog/hcatalog-pig-adapter/pom.xml 2f2e559 > hcatalog/server-extensions/pom.xml 985e9b1 > hcatalog/storage-handlers/hbase/pom.xml 4a17b3d > hcatalog/webhcat/java-client/pom.xml e1c738f > hcatalog/webhcat/svr/pom.xml 6f508c0 > hwi/pom.xml 5da6fc6 > itests/hcatalog-unit/pom.xml 97f53bb > itests/hive-unit/pom.xml c003437 > itests/qtest/pom.xml a453d8a > itests/test-serde/pom.xml 3f1f198 > itests/util/pom.xml 39e6c2e > jdbc/pom.xml b409956 > metastore/pom.xml f0089d6 > odbc/pom.xml 0ddf12d > packaging/pom.xml 3c2e5cb > packaging/src/main/assembly/bin.xml 5d8bb1a > pom.xml 70f4a4a > ql/pom.xml e4383a7 > serde/pom.xml 2031977 > service/pom.xml 3a0c7c6 > shims/0.20/pom.xml 9117824 > shims/0.20S/pom.xml 58bc6f4 > shims/0.23/pom.xml ebd0e5e > shims/assembly/pom.xml 48997d8 > shims/common-secure/pom.xml 12c102c > shims/common/pom.xml 5871b6a > testutils/pom.xml 59b1dcd > testutils/ptest2/pom.xml 00d2d6e > > Diff: https://reviews.apache.org/r/15497/diff/ > > > Testing > ------- > > Builds and installs in bootstrap mode. HiveQA will runs tests. > > > Thanks, > > Brock Noland > >
