Author: rmarroquin
Date: Thu Mar 20 09:08:39 2014
New Revision: 1579569

URL: http://svn.apache.org/r1579569
Log:
GORA-296

Modified:
    
gora/trunk/gora-hbase/src/main/java/org/apache/gora/hbase/store/HBaseStore.java

Modified: 
gora/trunk/gora-hbase/src/main/java/org/apache/gora/hbase/store/HBaseStore.java
URL: 
http://svn.apache.org/viewvc/gora/trunk/gora-hbase/src/main/java/org/apache/gora/hbase/store/HBaseStore.java?rev=1579569&r1=1579568&r2=1579569&view=diff
==============================================================================
--- 
gora/trunk/gora-hbase/src/main/java/org/apache/gora/hbase/store/HBaseStore.java 
(original)
+++ 
gora/trunk/gora-hbase/src/main/java/org/apache/gora/hbase/store/HBaseStore.java 
Thu Mar 20 09:08:39 2014
@@ -700,15 +700,17 @@ implements Configurable {
             keyClass.getCanonicalName())
             && classElement.getAttributeValue("name").equals(
                 persistentClass.getCanonicalName())) {
+          LOG.debug("Keyclass and nameclass match.");
 
           String tableNameFromMapping = 
classElement.getAttributeValue("table");
           String tableName = getSchemaName(tableNameFromMapping, 
persistentClass);
           
           //tableNameFromMapping could be null here
           if (!tableName.equals(tableNameFromMapping)) {
-            LOG.info("Keyclass and nameclass match but mismatching table names 
" 
-                + " mappingfile schema is '" + tableNameFromMapping 
-                + "' vs actual schema '" + tableName + "' , assuming they are 
the same.");
+          //TODO this might not be the desired behavior as the user might have 
actually made a mistake.
+            LOG.warn("Mismatching schema's names. Mappingfile schema: '" + 
tableNameFromMapping 
+                + "'. PersistentClass schema's name: '" + tableName + "'"
+                + "Assuming they are the same.");
             if (tableNameFromMapping != null) {
               mappingBuilder.renameTable(tableNameFromMapping, tableName);
             }
@@ -724,8 +726,6 @@ implements Configurable {
             mappingBuilder.addColumnFamily(tableName, family);
           }
           
-          
-          
           //we found a matching key and value class definition,
           //do not continue on other class definitions
           break;
@@ -788,5 +788,4 @@ implements Configurable {
     this.scannerCaching = numRows ;
     return this ;
   }
-  
-}
+}
\ No newline at end of file


Reply via email to