Author: [email protected]
Date: Thu Feb  2 15:53:44 2012
New Revision: 2055

Log:
AMDATU-523 Some renaming based on review comments

Added:
   
trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/Constants.java
      - copied, changed from r2054, 
/trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/ParameterTenantResolverConstants.java
   
trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/service/TenantRegistry.java
      - copied, changed from r2054, 
/trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/service/ParameterTenantMappingRegistry.java
Modified:
   
trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/ParameterTenantResolverConstants.java
   
trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/osgi/Activator.java
   
trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/service/ParameterTenantMappingRegistry.java
   
trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/service/ParameterTenantResolverExtenderFilter.java

Copied: 
trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/Constants.java
 (from r2054, 
/trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/ParameterTenantResolverConstants.java)
==============================================================================
--- 
/trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/ParameterTenantResolverConstants.java
      (original)
+++ 
trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/Constants.java
      Thu Feb  2 15:53:44 2012
@@ -19,6 +19,6 @@
  * @author <a href="mailto:[email protected]";>Amdatu Project 
Team</a>
  * 
  */
-public interface ParameterTenantResolverConstants {
+public interface Constants {
     String HOSTNAMES_KEY = "org.amdatu.tenantresolver.parameter";
 }
\ No newline at end of file

Modified: 
trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/osgi/Activator.java
==============================================================================
--- 
trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/osgi/Activator.java
 (original)
+++ 
trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/osgi/Activator.java
 Thu Feb  2 15:53:44 2012
@@ -21,7 +21,7 @@
 import org.amdatu.tenant.Tenant;
 import org.amdatu.web.dispatcher.DispatchExtenderFilter;
 import org.amdatu.web.dispatcher.DispatcherService;
-import 
org.amdatu.web.tenantresolver.parameter.service.ParameterTenantMappingRegistry;
+import org.amdatu.web.tenantresolver.parameter.service.TenantRegistry;
 import 
org.amdatu.web.tenantresolver.parameter.service.ParameterTenantResolverExtenderFilter;
 import org.apache.felix.dm.DependencyActivatorBase;
 import org.apache.felix.dm.DependencyManager;
@@ -40,7 +40,7 @@
     @Override
     public void init(BundleContext context, DependencyManager manager) throws 
Exception {
 
-        ParameterTenantMappingRegistry registry = new 
ParameterTenantMappingRegistry();
+        TenantRegistry registry = new TenantRegistry();
         ParameterTenantResolverExtenderFilter filter = new 
ParameterTenantResolverExtenderFilter(registry);
 
         Dictionary<String, Object> thProperties = new Hashtable<String, 
Object>();

Modified: 
trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/service/ParameterTenantResolverExtenderFilter.java
==============================================================================
--- 
trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/service/ParameterTenantResolverExtenderFilter.java
  (original)
+++ 
trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/service/ParameterTenantResolverExtenderFilter.java
  Thu Feb  2 15:53:44 2012
@@ -41,11 +41,11 @@
 
     public final static String TENANT_REQUEST_PARAMETER = "tenant";
 
-    private final ParameterTenantMappingRegistry m_registry;
+    private final TenantRegistry m_registry;
 
     private volatile LogService m_logService;
 
-    public 
ParameterTenantResolverExtenderFilter(ParameterTenantMappingRegistry registry) {
+    public ParameterTenantResolverExtenderFilter(TenantRegistry registry) {
         m_registry = registry;
     }
 

Copied: 
trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/service/TenantRegistry.java
 (from r2054, 
/trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/service/ParameterTenantMappingRegistry.java)
==============================================================================
--- 
/trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/service/ParameterTenantMappingRegistry.java
        (original)
+++ 
trunk/amdatu-web/tenantresolver-parameter/src/main/java/org/amdatu/web/tenantresolver/parameter/service/TenantRegistry.java
 Thu Feb  2 15:53:44 2012
@@ -21,15 +21,14 @@
 import org.amdatu.tenant.Tenant;
 
 /**
- * 
- * TODO document
- * TODO check for duplicates
- * TODO test
+ * {@code TenantRegistry) keeps a map of available {@link Tenant} services 
using their
+ * PID. This provides fasterer lookups and reduces contention on the service
+ * registry.
  * 
  * @author <a href="mailto:[email protected]";>Amdatu Project 
Team</a>
  * 
  */
-public final class ParameterTenantMappingRegistry {
+public final class TenantRegistry {
 
     // maps fast hostname lookup
     private final Map<String, Tenant> m_idToTenant =
_______________________________________________
Amdatu-commits mailing list
[email protected]
http://lists.amdatu.org/mailman/listinfo/amdatu-commits

Reply via email to