Author: ssmiweve
Date: 2007-06-01 13:10:31 +0200 (Fri, 01 Jun 2007)
New Revision: 5250

Modified:
   
branches/2.13/data-model-api/src/main/java/no/schibstedsok/searchportal/datamodel/generic/MapDataObjectSupport.java
   
branches/2.13/data-model-javabean-impl/src/main/java/no/schibstedsok/searchportal/datamodel/BeanDataNodeInvocationHandler.java
Log:
concurrency around dataObject properties. does not make datamodel synchronised 
but does ensure state safety across multiple-threads.

Modified: 
branches/2.13/data-model-api/src/main/java/no/schibstedsok/searchportal/datamodel/generic/MapDataObjectSupport.java
===================================================================
--- 
branches/2.13/data-model-api/src/main/java/no/schibstedsok/searchportal/datamodel/generic/MapDataObjectSupport.java
 2007-06-01 10:38:44 UTC (rev 5249)
+++ 
branches/2.13/data-model-api/src/main/java/no/schibstedsok/searchportal/datamodel/generic/MapDataObjectSupport.java
 2007-06-01 11:10:31 UTC (rev 5250)
@@ -76,7 +76,7 @@
 
     public V getValue(final String key){
 
-        return map.get(key);
+        return null != key ? map.get(key) : null;
     }
 
     public void setValue(final String key, final V value){

Modified: 
branches/2.13/data-model-javabean-impl/src/main/java/no/schibstedsok/searchportal/datamodel/BeanDataNodeInvocationHandler.java
===================================================================
--- 
branches/2.13/data-model-javabean-impl/src/main/java/no/schibstedsok/searchportal/datamodel/BeanDataNodeInvocationHandler.java
      2007-06-01 10:38:44 UTC (rev 5249)
+++ 
branches/2.13/data-model-javabean-impl/src/main/java/no/schibstedsok/searchportal/datamodel/BeanDataNodeInvocationHandler.java
      2007-06-01 11:10:31 UTC (rev 5250)
@@ -101,7 +101,7 @@
             return super.invoke(obj, method, args);
 
         } catch (IllegalArgumentException iae) {
-            LOG.debug("property not one of our own", iae);
+            LOG.debug("property not one of our own. " + iae.getMessage());
         }
 
         // try non-(dataObject|dataNode) delegated-properties

_______________________________________________
Kernel-commits mailing list
[email protected]
http://sesat.no/mailman/listinfo/kernel-commits

Reply via email to