Author: ssmiweve
Date: 2008-04-15 14:31:19 +0200 (Tue, 15 Apr 2008)
New Revision: 6445

Modified:
   
branches/2.17/data-model-api/src/main/java/no/sesat/search/datamodel/DataModelFactory.java
   branches/2.17/site-spi/src/main/java/no/sesat/search/site/Site.java
   
branches/2.17/site-spi/src/main/java/no/sesat/search/site/config/SiteConfiguration.java
   
branches/2.17/war/src/main/java/no/sesat/search/http/filters/SiteLocatorFilter.java
Log:
better exception handling related to work going on in OPS-209

Modified: 
branches/2.17/data-model-api/src/main/java/no/sesat/search/datamodel/DataModelFactory.java
===================================================================
--- 
branches/2.17/data-model-api/src/main/java/no/sesat/search/datamodel/DataModelFactory.java
  2008-04-15 12:17:20 UTC (rev 6444)
+++ 
branches/2.17/data-model-api/src/main/java/no/sesat/search/datamodel/DataModelFactory.java
  2008-04-15 12:31:19 UTC (rev 6445)
@@ -186,7 +186,7 @@
 
             final String clsName = 
siteConf.getProperty(DATA_MODEL_FACTORY_IMPL);
             LOG.info("constructing for " + cxt.getSite() + " instance of " + 
clsName);
-            
+                    
             @SuppressWarnings("unchecked")
             final Class<DataModelFactory> cls = (Class<DataModelFactory>) 
Class.forName(clsName);
 

Modified: branches/2.17/site-spi/src/main/java/no/sesat/search/site/Site.java
===================================================================
--- branches/2.17/site-spi/src/main/java/no/sesat/search/site/Site.java 
2008-04-15 12:17:20 UTC (rev 6444)
+++ branches/2.17/site-spi/src/main/java/no/sesat/search/site/Site.java 
2008-04-15 12:31:19 UTC (rev 6445)
@@ -31,6 +31,7 @@
 import java.util.Properties;
 import java.util.concurrent.locks.ReentrantReadWriteLock;
 import no.schibstedsok.commons.ioc.BaseContext;
+import org.apache.log4j.Level;
 import org.apache.log4j.Logger;
 
 /** A Site object idenetifies an unique SiteSearch implementation.
@@ -274,6 +275,9 @@
             LOG.fatal(FATAL_CANT_FIND_DEFAULT_SITE, ex);    
         }
         
+        final Level oLevel = LOG.getLevel();
+        LOG.setLevel(Level.ALL);
+        
         final String defaultSiteName = props.getProperty(DEFAULT_SITE_KEY, 
System.getProperty(DEFAULT_SITE_KEY));
         LOG.info("defaultSiteName: " + defaultSiteName);
         
@@ -285,6 +289,7 @@
                 = props.getProperty(DEFAULT_SERVER_PORT_KEY, 
System.getProperty(DEFAULT_SERVER_PORT_KEY));
         LOG.info("defaultSitePort: " + defaultSitePort);
         
+        LOG.setLevel(oLevel);
         
         SERVER_PORT = Integer.parseInt(defaultSitePort);
 

Modified: 
branches/2.17/site-spi/src/main/java/no/sesat/search/site/config/SiteConfiguration.java
===================================================================
--- 
branches/2.17/site-spi/src/main/java/no/sesat/search/site/config/SiteConfiguration.java
     2008-04-15 12:17:20 UTC (rev 6444)
+++ 
branches/2.17/site-spi/src/main/java/no/sesat/search/site/config/SiteConfiguration.java
     2008-04-15 12:31:19 UTC (rev 6445)
@@ -90,6 +90,11 @@
             cxt.newPropertiesLoader(cxt, Site.CONFIGURATION_FILE, 
properties).abut();
 
             INSTANCES.put(cxt.getSite(), this);
+            
+        }catch(ResourceLoadException rle){
+            LOG.fatal("BROKEN SITE HIERARCHY." + rle.getMessage());
+            throw new VirtualMachineError(rle.getMessage()){};
+            
         } finally {
             INSTANCES_LOCK.writeLock().unlock();
         }

Modified: 
branches/2.17/war/src/main/java/no/sesat/search/http/filters/SiteLocatorFilter.java
===================================================================
--- 
branches/2.17/war/src/main/java/no/sesat/search/http/filters/SiteLocatorFilter.java
 2008-04-15 12:17:20 UTC (rev 6444)
+++ 
branches/2.17/war/src/main/java/no/sesat/search/http/filters/SiteLocatorFilter.java
 2008-04-15 12:31:19 UTC (rev 6445)
@@ -35,6 +35,8 @@
 import java.util.concurrent.TimeUnit;
 import java.util.concurrent.locks.Lock;
 import java.util.concurrent.locks.ReentrantLock;
+import javax.enterprise.deploy.model.J2eeApplicationObject;
+import javax.enterprise.deploy.model.J2eeApplicationObject;
 import javax.servlet.Filter;
 import javax.servlet.FilterChain;
 import javax.servlet.FilterConfig;
@@ -53,6 +55,7 @@
 import no.sesat.search.site.config.UrlResourceLoader;
 import no.sesat.search.site.Site;
 import no.sesat.search.datamodel.DataModel;
+import no.sesat.search.site.config.ResourceLoadException;
 import no.sesat.search.view.FindResource;
 import org.apache.commons.lang.StringEscapeUtils;
 import org.apache.commons.lang.time.StopWatch;
@@ -105,8 +108,14 @@
             final Properties props = new Properties();
             final PropertiesLoader loader
                     = UrlResourceLoader.newPropertiesLoader(siteContext, 
Site.CONFIGURATION_FILE, props);
-            loader.abut();
-            return props.getProperty(Site.PARENT_SITE_KEY);
+            try{
+                loader.abut();
+                return props.getProperty(Site.PARENT_SITE_KEY);
+                
+            }catch(ResourceLoadException rle){
+                LOG.fatal("BROKEN SITE HIERARCHY." + rle.getMessage());
+                throw new VirtualMachineError(rle.getMessage()){};
+            }
         }
     };
 

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

Reply via email to