[ https://issues.apache.org/jira/browse/HBASE-6087?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13285979#comment-13285979 ]
stack commented on HBASE-6087: ------------------------------ Why this change in a patch that adds a new module? {code} +# For releases, add hbase & webapps to CLASSPATH +# Webapps must come first else it messes up Jetty +if [ -d "$HBASE_HOME/hbase-webapps" ]; then + add_to_cp_if_exists "${HBASE_HOME}" +fi +#add if we are in a dev environment +if [ -d "$HBASE_HOME/hbase-server/target/hbase-webapps" ]; then + add_to_cp_if_exists "${HBASE_HOME}/hbase-server/target" +fi {code} Why is it being moved earlier? Doesn't this get inherited from parent? "+ <version>0.95-SNAPSHOT</version>" (This is version of the new hbase-common pom.xml). This comment is wrong? {code} + <!-- + profile for building against Hadoop 0.23.0. Activate using: + mvn -Dhadoop.profile=23 + --> {code} Thats good HBaseConfiguration comes up. HConstants too? That think is an ugly anti-pattern. I'm fine w/ it going up but need to figure how to kill it. I'm +1 on committing this. Good stuff Jesse. Is it ready to go you think? > Add hbase-common module > ----------------------- > > Key: HBASE-6087 > URL: https://issues.apache.org/jira/browse/HBASE-6087 > Project: HBase > Issue Type: Bug > Components: build > Affects Versions: 0.96.0 > Reporter: Jesse Yates > Assignee: Jesse Yates > Attachments: hbase-6087-v0.patch, hbase-6087-v1.patch > > > Add an hbase-common module so common/utility classes can be pulled up out of > hbase-server. This is _not_ the moving of classes, just the general project > setup. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira