-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/6494/
-----------------------------------------------------------
(Updated Aug. 31, 2012, 10:34 p.m.)
Review request for hcatalog and Francis Liu.
Changes
-------
Hey Francis, here's the updated patch against trunk.
Notice how server-extensions uses maven-ant-tasks for dependency management,
rather than ivy. My experience with these maven tasks is generally positive and
I'd like to migrate the rest of our subprojects over time. It gives us full
control of the pom file we'll use when publishing to the central repo too.
Lastly, if we ever choose to use maven as our build system we already have
these pom files, which makes experimenting with maven easy in a branch.
Description
-------
hcatalog-server-extensions.jar is an existing artifact we provide. This change
refactors the repo layout & build so that server-extensions is a new subproject
with its own Ivy dependencies.
The benefit is users who do not need the server extensions do not pull JMS &
other transitive dependencies into their project.
This addresses bug HCATALOG-461.
https://issues.apache.org/jira/browse/HCATALOG-461
Diffs (updated)
-----
ant/deploy.xml 84c3648
build-common.xml ac9b4f5
build.xml 76b8cb4
hcatalog-pig-adapter/build.xml 8dbc319
ivy.xml 11e45bf
server-extensions/build.xml PRE-CREATION
server-extensions/pom.xml PRE-CREATION
server-extensions/src/main/java/org/apache/hcatalog/listener/NotificationListener.java
PRE-CREATION
server-extensions/src/test/java/org/apache/hcatalog/listener/TestMsgBusConnection.java
PRE-CREATION
server-extensions/src/test/java/org/apache/hcatalog/listener/TestNotificationListener.java
PRE-CREATION
src/java/org/apache/hcatalog/listener/NotificationListener.java 8529c5a
src/test/org/apache/hcatalog/listener/TestMsgBusConnection.java 4da3cef
src/test/org/apache/hcatalog/listener/TestNotificationListener.java 7d09d79
webhcat/java-client/build.xml fc1386c
webhcat/svr/build.xml a7bc34c
Diff: https://reviews.apache.org/r/6494/diff/
Testing
-------
Thanks,
Travis Crawford
