[ 
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

        

Reply via email to