-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7380/
-----------------------------------------------------------

(Updated Oct. 1, 2012, 11:19 p.m.)


Review request for hcatalog.


Changes
-------

Fix small issue with previous patch.


Description
-------

Update the following subprojects to maven-ant-tasks for dependency resolution 
(removing ivy):

* hcatalog-pig-adapter
* webhcat
* webhcat-java-client

Fix an issue with the top-level POM file where it contained a nested "project" 
element (invalid syntax).

Improve the top-level "jar" target so it builds all jars in that target, rather 
than scattering around different targets.

Remove either unused or duplicate property definitions. This is a major area 
for improvement because it really introduces unnecessary confusion. Unused 
property suggest certain values are configurable when in fact they are not, and 
duplicate definitions confuse users attempting to understand what a value it 
set to. This is an incremental step towards moving as many properties as is 
reasonable to build.properties. When that has been achieved developers will 
have a single place to see what parts of the build are configurable.


This addresses bug HCATALOG-519.
    https://issues.apache.org/jira/browse/HCATALOG-519


Diffs (updated)
-----

  build.xml 1b18d70 
  hcatalog-pig-adapter/build.xml a6de0b3 
  hcatalog-pig-adapter/ivy.xml f0341c5 
  hcatalog-pig-adapter/pom.xml PRE-CREATION 
  pom.xml 5402176 
  storage-handlers/hbase/build.xml 6e7d635 
  webhcat/java-client/build.xml c47db0f 
  webhcat/java-client/ivy.xml 7b048c5 
  webhcat/java-client/pom.xml PRE-CREATION 
  webhcat/svr/build.xml ed7ecf0 
  webhcat/svr/ivy.xml 9a5e37f 
  webhcat/svr/pom.xml PRE-CREATION 

Diff: https://reviews.apache.org/r/7380/diff/


Testing
-------

Built & tested locally, as well as CI.

https://travis.ci.cloudbees.com/job/HCATALOG-519_pig_adapter_pom/


Thanks,

Travis Crawford

Reply via email to