Add ResponseTime annotation to Service

project-open-data · 9577669e520ef3cf78f2c25ba61c4082c51759b2 · jmarin committed Sep 29, 2012 · Sep 29, 2012 · 9577669
1 parent c431718
commit 9577669e520ef3cf78f2c25ba61c4082c51759b2
Unified Split

Showing with 17 additions and 18 deletions.

+4 −8 src/main/java/org/geo/spatialsearch/rest/api/impl/CensusResourceImpl.java

+13 −10 src/main/java/org/geo/spatialsearch/service/impl/CensusServiceImpl.java
diff --git a/src/main/java/org/geo/spatialsearch/rest/api/impl/CensusResourceImpl.java b/src/main/java/org/geo/spatialsearch/rest/api/impl/CensusResourceImpl.java
@@ -43,12 +43,9 @@
     @Qualifier("exceptionHandler")
     private ExceptionHandler handler;
 
-    // private static final String CENSUS_BY_COORDINATES =
-    // "Census-findByCoordinates";
-    // private static final String CENSUS_BY_FIPSCODE =
-    // "Census-findGeographyByFips";
-    // private static final String CENSUS_BY_GEOGRAPHY_NAME =
-    // "Census-findGeographyByName";
+//    private static final String CENSUS_BY_COORDINATES = "Census-findByCoordinates";
+//    private static final String CENSUS_BY_FIPSCODE = "Census-findGeographyByFips";
+//    private static final String CENSUS_BY_GEOGRAPHY_NAME = "Census-findGeographyByName";
 
     @Override
     @Path(value = "{geography}")
@@ -71,8 +68,7 @@ public Response findByCoordinates(@Context UriInfo uriInfo,
             exception = ex;
             // System.out.println(ex.getStackTrace());
         }
-        // APIStatsProfiler.captureStatistics(CENSUS_BY_COORDINATES,
-        // apiResponse, uriInfo, true, exception);
+        //APIStatsProfiler.captureStatistics(CENSUS_BY_COORDINATES, apiResponse,uriInfo, true, exception);
         Response response = RestFormatUtil
                 .format(format, callback, apiResponse);
         return response;
diff --git a/src/main/java/org/geo/spatialsearch/service/impl/CensusServiceImpl.java b/src/main/java/org/geo/spatialsearch/service/impl/CensusServiceImpl.java
@@ -14,6 +14,7 @@
 import org.geo.spatialsearch.service.CensusService;
 import org.geo.spatialsearch.util.GeometryUtil;
 import org.geo.spatialsearch.util.Message;
+import org.geo.spatialsearch.util.ResponseTime;
 import org.geo.spatialsearch.util.ValidationUtil;
 import org.hibernate.Criteria;
 import org.hibernate.spatial.criterion.SpatialRestrictions;
@@ -32,6 +33,7 @@
 
     @Override
     @Transactional(readOnly = true)
+    @ResponseTime
     public CensusGeoBaseObject findById(CensusGeographyEnum geographyType,
             Long id) {
         CensusGeoBaseObject geoEntity = null;
@@ -45,6 +47,7 @@ public CensusGeoBaseObject findById(CensusGeographyEnum geographyType,
 
     @Override
     @Transactional(readOnly = true)
+    @ResponseTime
     public CensusLookupResponse findByCoordinates(
             CensusGeographyEnum geographyType, double longitude, double latitude) {
 
@@ -98,55 +101,55 @@ public CensusLookupResponse findByCoordinates(
 
     private void findAllByPoint(CensusLookupResponse apiResponse, Point point) {
         // TODO Auto-generated method stub
-        
+
     }
 
     private void findTribalByPoint(CensusLookupResponse apiResponse, Point point) {
         // TODO Auto-generated method stub
-        
+
     }
 
     private void findStateSenateDistrictByPoint(
             CensusLookupResponse apiResponse, Point point) {
         // TODO Auto-generated method stub
-        
+
     }
 
     private void findStateHouseDistrictByPoint(
             CensusLookupResponse apiResponse, Point point) {
         // TODO Auto-generated method stub
-        
+
     }
 
     private void findPlaceByPoint(CensusLookupResponse apiResponse, Point point) {
         // TODO Auto-generated method stub
-        
+
     }
 
     private void findMSAByPoint(CensusLookupResponse apiResponse, Point point) {
         // TODO Auto-generated method stub
-        
+
     }
 
     private void findCongressionalDistrictByPoint(
             CensusLookupResponse apiResponse, Point point) {
         // TODO Auto-generated method stub
-        
+
     }
 
     private void findBlockByPoint(CensusLookupResponse apiResponse, Point point) {
         // TODO Auto-generated method stub
-        
+
     }
 
     private void findTractByPoint(CensusLookupResponse apiResponse, Point point) {
         // TODO Auto-generated method stub
-        
+
     }
 
     private void findCountyByPoint(CensusLookupResponse apiResponse, Point point) {
         // TODO Auto-generated method stub
-        
+
     }
 
     @Transactional(readOnly = true)